-
Notifications
You must be signed in to change notification settings - Fork 22
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
Rename eon functions #281
Rename eon functions #281
Conversation
490be69
to
4f89f95
Compare
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.
I like the new naming 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.
💯 just one comment however
@@ -125,19 +134,19 @@ forEraInEon :: () | |||
forEraInEon era no yes = | |||
inEonForEra no yes era | |||
|
|||
inEraEonMaybe :: () | |||
forEraInEonMaybe :: () |
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.
👍
4f89f95
to
4b6ecd8
Compare
…p-text Fix typo in stake-pool help text and clarify drep queries arguments
Changelog
Context
The function names are composed on smaller positional components:
forEra
- corresponds toCardanoEra era
forShelleyBasedEra
- corresponds toShelleyBasedEra era
inEon
- corresponds toa
andeon era -> a
if not coupled withmaybe
otherwise corresponds to justeon era -> a
maybe - corresponds to
Maybe a`maybeEon
- corresponds toMaybe (eon era)
Checklist
See Running tests for more details
.cabal
files are updatedhlint
. See.github/workflows/check-hlint.yml
to get thehlint
versionstylish-haskell
. See.github/workflows/stylish-haskell.yml
to get thestylish-haskell
versionghc-8.10.7
andghc-9.2.7