Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Newer Servant #5

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

Newer Servant #5

wants to merge 5 commits into from

Conversation

Fresheyeball
Copy link

Support for newer versions of Servant
Nix support

Isaac Shapira and others added 5 commits August 9, 2018 15:20
…allow custom port for toxiproxy (#1)

* pin nixpkgs rev

* allow custom port for toxiproxy

* export BaseUrl
@414owen
Copy link

414owen commented Jan 31, 2024

@jpittis ping, we're getting compile errors trying to use this package. Looks like this PR implements the fix.

@jpittis
Copy link
Owner

jpittis commented Feb 25, 2024

Unfortunately I'm in a situation where I haven't written any Haskell since this project was posted, and I'm not sufficiently familiar with nix to review the contents of this pull request.

If one of you is using this repository for something serious with a meaningful maintenance horizon, can I suggest that you fork the repo and take over maintenance via the fork.

I'd be willing to merge this PR if it didn't include the nix additions as the code changes look good enough for me.

@414owen
Copy link

414owen commented Mar 20, 2024

@Fresheyeball gentle ping, how would you feel about splitting the nix and haskell changes?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants