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

alex/update ghc #11

Merged
merged 3 commits into from
Jan 6, 2024
Merged

alex/update ghc #11

merged 3 commits into from
Jan 6, 2024

Conversation

alexbiehl
Copy link
Contributor

  • Allow building with recent GHCs
  • Test recent GHCs during CI

Copy link
Owner

@phile314 phile314 left a comment

Choose a reason for hiding this comment

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

Mostly LGTM.

with:
cabal-version: latest
ghc-version: ${{ matrix.ghc }}
- run: cabal v2-test
- run: cabal v2-build
Copy link
Owner

Choose a reason for hiding this comment

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

Doesn't this mean the tests are no longer run at all? I mean it's debatable if running the tests should be a separate step, but they should definitely be run at some point.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Not sure why I changed this to be build only 🤔 maybe it’d fail calling cabal test on a package that doesn’t have tests.

@LaurentRDC
Copy link

@alexbiehl Gentle ping to address the changes requested above. Your PR would allow users to use pinch-gen with GHC 9.8!

@phile314 phile314 merged commit 9b6ecb5 into phile314:master Jan 6, 2024
9 checks passed
@alexbiehl alexbiehl deleted the alex/update-ghc branch January 6, 2024 18:36
@phile314
Copy link
Owner

phile314 commented Jan 7, 2024

Merged and released as 0.4.4.0.

@LaurentRDC
Copy link

@phile314 Thank you for making a release!

Would you be able to update the Hackage metadata to allow base-4.19 (GHC 9.8)? I can already use pinch-gen with GHC 9.8 using the following clause in a cabal.project file:

allow-newer: pinch-gen:base

@phile314
Copy link
Owner

phile314 commented Jan 8, 2024

I took out the sledge hammer and increased the base upper limit to 5 😄

Let's see how this goes for the next few GHC releases.

@LaurentRDC
Copy link

I greatly appreciate this, thank you!

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