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

Miscellaneous fixes #2539

Merged
merged 3 commits into from
Nov 12, 2021
Merged

Miscellaneous fixes #2539

merged 3 commits into from
Nov 12, 2021

Conversation

nc6
Copy link
Contributor

@nc6 nc6 commented Nov 5, 2021

Each commit can be reviewed independently.

nc6 added 2 commits November 5, 2021 12:29
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'
```
@nc6 nc6 requested a review from lehins November 5, 2021 11:37
@nc6 nc6 requested a review from TimSheard as a code owner November 5, 2021 11:37
Copy link
Collaborator

@lehins lehins left a 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!

libs/cardano-ledger-core/src/Cardano/Ledger/Hashes.hs Outdated Show resolved Hide resolved
@lehins
Copy link
Collaborator

lehins commented Nov 5, 2021

Copy link
Contributor

@JaredCorduan JaredCorduan left a 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.
@nc6
Copy link
Contributor Author

nc6 commented Nov 11, 2021

Turns out we don't use the nix specified version in CI, so I've downbumped it to the 3.1.0 release

@nc6 nc6 merged commit ae89eb9 into master Nov 12, 2021
@nc6 nc6 deleted the nc/misc-fixes branch November 12, 2021 20:57
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