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

Simplification of Nix flake and removal of GHC 8 #245

Merged
merged 14 commits into from
Oct 1, 2024

Conversation

DavidMazarro
Copy link
Member

@DavidMazarro DavidMazarro commented Sep 21, 2024

This PR simplifies the Nix flake and GHC versions of the flake by removing GHC 8. It also:

  • Bumps GHC 9 version from 9.0.2 to 9.6.6 and transformers dependency upper bound.
  • Changes the GHC-musl image to the one mentioned here, since the previous one we were using seems to be unmaintained.
  • Adds Cabal, HLint and HLS to the development shell.
  • Adds a .envrc file for use with direnv.
  • Updates flake.lock.

@DavidMazarro DavidMazarro self-assigned this Sep 21, 2024
Also changed the maintainer to Cristhian Motoche
<cmotoche@stackbuilders.com>
The previous one we were using seems to be unmaintained
@DavidMazarro DavidMazarro marked this pull request as ready for review September 30, 2024 10:53
Dockerfile Show resolved Hide resolved
flake.nix Outdated Show resolved Hide resolved
hapistrano.cabal Outdated Show resolved Hide resolved
DavidMazarro and others added 4 commits October 1, 2024 10:32
Co-authored-by: Sebastián Estrella <2049686+sestrella@users.noreply.github.com>
Co-authored-by: Sebastián Estrella <sestrella@stackbuilders.com>
Since we now only expose one in the flake
@dalejo96 dalejo96 requested a review from sestrella October 1, 2024 14:39
@sestrella sestrella merged commit e910902 into main Oct 1, 2024
4 checks passed
@sestrella sestrella deleted the DM/simplify-nix-flake-ghc-versions branch October 1, 2024 15:14
@sestrella
Copy link
Member

@DavidMazarro nice job!

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