-
Notifications
You must be signed in to change notification settings - Fork 286
Upgrade to LTS 22.26 GHC 9.6.5 #5142
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
Changes from all commits
Commits
Show all changes
34 commits
Select commit
Hold shift + click to select a range
2d1bf19
Mostly Upgrade LTS to ghc 9.6.5
ChrisPenner 61b2666
Temporarily ignore deprecations from jose
ChrisPenner 4a9b358
Swap from x509 to crypton-x509 since old packages are abandoned
ChrisPenner 61e0522
Fix redundant liftA2 imports
ChrisPenner 745660d
Fix servant request stuff
ChrisPenner a7f234a
Use random token rather than NanoID
ChrisPenner f3d4455
Update opt-parse help
ChrisPenner 7538111
Upgrade watch expression stuff
ChrisPenner b85bea8
Clean up stack.yaml pins
ChrisPenner 87e15d7
Bump stack version in CI
ChrisPenner 06e730d
Attempt to bump nix flake
ChrisPenner c97d93e
lsp 2.3.0.0
neduard a35d706
Use GHC.IsList
neduard 95a0eb9
cleanup cabal
neduard c3fe983
Merge neduard's lsp fixes
ChrisPenner 5dcc096
update transcripts
ChrisPenner e94f870
Remove Primes from module names
ChrisPenner 563e148
Resolve conflicts with trunk
ChrisPenner 8f694f1
Add note about deprecations
ChrisPenner 15f236a
Add issue number
ChrisPenner 121d51c
Don’t export packages for individual build tools
sellout d0f4c70
Remove the non-haskell.nix devShell
sellout 01e0351
Switch `nixos-unstable` to `release-23.11`
sellout a5b9864
Don’t hide `unison-project` in an overlay
sellout ba13de4
Remove dead code
sellout 86819b8
Remove reference to non-existent Cabal component
sellout 110b5ca
Actually cache UCM in CI
sellout f25af69
Override tool versions directly in the package set
sellout dad9d6d
Minor simplification of haskell-nix-flake
sellout 7b373d7
Eliminate `only-tools` devShell
sellout f2c8020
Can now include the devShells in `all` package
sellout 5ae7963
Merge branch 'simplifying-nix' into cp/ghc-upgrade
sellout c47bdcf
Get Nix build working with GHC 9.6.5
sellout ecf5fe1
Merge pull request #5155 from sellout/cp/ghc-upgrade
aryairani File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
we can also change the default on
unisonweb/actions/stack/installif we like that better.There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah, I figured it was a bit of a chicken-and-egg thing. TBH I kind of like specifying here because it's easy to see what it's using and easy to bump again in the future 😄
I'm glad you added it as an arg for that step.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah, I like specifying it in this project too; I just wish we didn't have to specify it 4 times in this project, but maybe that's fine. Replace All.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You can read JSON in GH workflows, and also in Nix (I don’t know if there are other places we need versions).
So, we could write a
dependency-versions.jsonfile or something that the flake can read into itsversionsattrSet, and that workflows can read as well. I’ve been wanting to extract it from the flake anyway.