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

Allow aeson 1.5 #1537

Merged
merged 1 commit into from
May 26, 2020
Merged

Allow aeson 1.5 #1537

merged 1 commit into from
May 26, 2020

Conversation

felixonmars
Copy link
Contributor

Builds fine and all tests pass here.

Builds fine and all tests pass here.
@steve-chavez
Copy link
Member

LGTM 👍

@steve-chavez steve-chavez merged commit 784ebe5 into PostgREST:master May 26, 2020
@robx
Copy link
Contributor

robx commented Jun 7, 2020

Thanks for this! The relevant stackage tracking ticket: commercialhaskell/stackage#5393

@steve-chavez I wonder if we could make a cabal revision with this change on hackage relaxing this bound in version 7.0.1, since no source-level change seems involved?

@steve-chavez
Copy link
Member

Hey @robx. You mean Hackage Metadata Revisions right? I didn't know it was possible to update the dependencies bounds without a new release. I'm reading those docs now...(still not clear how to do the revision).

@steve-chavez
Copy link
Member

steve-chavez commented Jun 7, 2020

Argh, I'm not seeing how to do the revision. @robx Do you have a guide on how to do it?

Now I see its done through the hackage UI. I'll do the change.

@robx
Copy link
Contributor

robx commented Jun 7, 2020

Great, thanks! Haven't done this myself before, but happy to help try figure it out if it doesn't work.

@steve-chavez
Copy link
Member

@robx Done! See: https://hackage.haskell.org/package/postgrest-7.0.1/revisions. Let me know if the change is good.

@robx
Copy link
Contributor

robx commented Jun 7, 2020

Looks perfect, thanks!

monacoremo pushed a commit to monacoremo/postgrest that referenced this pull request Jul 17, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants