-
Notifications
You must be signed in to change notification settings - Fork 156
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
Miscellaneous fixes #2539
Miscellaneous fixes #2539
Conversation
This is useful to compute the hash for dependencies when bumping in cabal.project. In particular using this git alias (from @redxaxder): ``` [alias] nix = !nix-prefetch-git \"file://$(git rev-parse --show-toplevel)\" --rev \"$(git rev-parse HEAD)\" 2>&1 | grep 'sha256\\|rev\"' | sed 's/\"rev\"/tag/g' | sed 's/\"sha256\"/--sha256/g' | sed 's/[,\"]//g' ```
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.
Just one comment, otherwise looks great!
I think you also need to update ormolu in github actions https://github.com/input-output-hk/cardano-ledger/blob/4347e54b2827090e898702ccfb93cc731b4a1909/.github/workflows/haskell.yml#L145 |
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.
thanks @nc6 !
The ormolu version in use was quite old. This commit bumps it to later and reformats. I have eyeballed the changes and think they're generally improvements. They mostly fit into three groups: - The expression `(x -n)` is now formatted as `(x - n)`. - A space is enforced after the -- in a comment.
Turns out we don't use the nix specified version in CI, so I've downbumped it to the 3.1.0 release |
Each commit can be reviewed independently.