Skip to content

Refactor servant-client-core #1130

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

Merged
merged 2 commits into from
Feb 18, 2019

Conversation

phadej
Copy link
Contributor

@phadej phadej commented Feb 18, 2019

  • No more Internal modules
  • Remove ClientLike-generic. Let's use Routes-generics
    • Let's see if anyone notices, otherwise we can add it back (there were no cookbook, neither tutorial breakages!)
  • Add Makefile for common tasks
    • Fix servant-client-ghcjs

- No more Internal modules
- Remove ClientLike-generic. Let's use Routes-generics
    - Let's see if anyone notices, otherwise we can add it back
- Add Makefile for common tasks
    - Fix servant-client-ghcjs
@phadej phadej merged commit e922b98 into haskell-servant:master Feb 18, 2019
@phadej phadej deleted the refactor-servant-client branch February 18, 2019 18:09
@phadej phadej added this to the 0.16 milestone Feb 18, 2019
@parsonsmatt
Copy link
Contributor

Now that we're finally upgraded to Stackage lts-14, I've been bitten by the missing ClientLike stuff! Consider it noticed 😉 Can you point me to a migration path to the Routes-generics you mention? Thanks!

@phadej
Copy link
Contributor Author

phadej commented Jan 24, 2020

@parsonsmatt
Copy link
Contributor

Thanks! i think I can figure it out from there.

@parsonsmatt
Copy link
Contributor

OK, I was not able to figure it out, and ripped out all the generic stuff and reverted to boring code. We used nested clients in a way that I couldn't make work. Is there a blocker towards rereleasing the old generic ClientLIke code?

@phadej
Copy link
Contributor Author

phadej commented Jan 24, 2020 via email

@parsonsmatt
Copy link
Contributor

No worries, I completely understand not wanting to maintain it :)

Apologies for 'boring', didn't mean anything negative by it.

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.

2 participants