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

Support new conduit release #773

Merged
merged 25 commits into from
Feb 1, 2018
Merged

Support new conduit release #773

merged 25 commits into from
Feb 1, 2018

Conversation

snoyberg
Copy link
Member

The new version of conduit (1.3) is going to be released soon, and will be switching from monad-control over to unliftio-core. This PR moves the persistent API over as well for compatibility, while still keeping compatibility with the older version (though Travis may discover a few things I still need to tweak to make this work). It also drops usage of deprecated operators and type synonyms in conduit 1.3, and cleans up a few other warnings.

Is there any objection to me merging this and making a new major release of persistent, persistent-sqlite, persistent-postgresql, persistent-mysql, and persistent-mongoDB when I'm releasing the new version of conduit?

@MaxGabriel
Copy link
Member

I'm good with that. There are a few PRs that were waiting on a breaking change for their release:

@snoyberg
Copy link
Member Author

Alright, I've merged both #760 and #734 into this branch, and I've reverted #759. Can you mind giving those last commits a review to ensure I hit what you were looking for? Also, CC @parsonsmatt.

@@ -1,3 +1,7 @@
## 2.8.0

* Switch from `MonadBaseControl` to `MonadUnliftIO`
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🙌 🎉

@MaxGabriel
Copy link
Member

Didn’t look over too closely but it LGTM

@MaxGabriel
Copy link
Member

MaxGabriel commented Jan 17, 2018

I think something was going wrong with the Travis build on this (Travis's UI said it had been running for over 6 hours without any progress) so I killed it. Edit: I restarted it

@parsonsmatt
Copy link
Collaborator

I'm trying to use this latest commit for testing/updating persistent-typed-db and it's failing to build. Probably need to add conduit-1.3.0 into the stack.yaml?

@snoyberg
Copy link
Member Author

I've been testing with a megarepo that includes all the deps, since I've been modifying all of the repos in lock step. The new conduit version isn't yet on Hackage. I've pushed a commit that adds appropriate extra-deps to hopefully get this compiling.

@ghost
Copy link

ghost commented Jan 27, 2018

When I want to clone this repo ,to make the drop ghc part, how is this branch named.

@MaxGabriel
Copy link
Member

@ghost it’s named newer-conduit

@snoyberg snoyberg merged commit b7f0194 into master Feb 1, 2018
@snoyberg snoyberg deleted the newer-conduit branch February 1, 2018 21:38
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.

None yet

3 participants