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

bump network to 3.2.* and data-default to 0.8.* #41

Merged
merged 3 commits into from
Feb 15, 2025

Conversation

RubenAstudillo
Copy link
Contributor

@RubenAstudillo RubenAstudillo commented Feb 4, 2025

This is a partner PR for the same HaskellNet PR . There is a network 3.2.7 and data-default 0.8 published on hackage, they both compile cleanly on this repository. The module structure did not change enough to make this project not compile. I am bumping the version on both of them.

Supersedes #40

@mpscholten
Copy link

@MangoIV could you take a look at this?

@MangoIV
Copy link
Collaborator

MangoIV commented Feb 15, 2025

I guess because HaskellNet's PR is not merged this is not really testing the new network at all.

Do you think merging before that is merged and released makes sense / is required?

@MangoIV MangoIV mentioned this pull request Feb 15, 2025
@mpscholten
Copy link

True, but merging this will allow one less local override / nixpkgs jailbreak. So as long as you don't see any downsides I'd suggest going ahead 👍

@MangoIV
Copy link
Collaborator

MangoIV commented Feb 15, 2025

Hmm okay, since I don’t expect a major version bump on HaskellNet for your PR, I’ll merge this.

@MangoIV MangoIV merged commit 5379ecf into dpwright:master Feb 15, 2025
10 checks passed
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.

3 participants