-
Notifications
You must be signed in to change notification settings - Fork 155
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
Conway Genesis PParams #3499
Conway Genesis PParams #3499
Conversation
193fb95
to
d7b462f
Compare
c636523
to
c610a9a
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.
Overall direction is good. Needs:
- a rebase on the PParams implementation
- inside JSON hiding the fact that Genesis contains UpgradePParams
- Creating a pattern synonym that hides the fact that Genesis contains UpgradePParams. See AlonzoGenesis
f4dce00
to
1adbfce
Compare
d7b462f
to
3d30872
Compare
91199f1
to
69b02e8
Compare
69b02e8
to
b656698
Compare
8afba03
to
bc895b7
Compare
8dee5f0
to
2dd1826
Compare
d4770d2
to
7b9bb85
Compare
6b81843
to
19b04de
Compare
d0be5d9
to
7b20016
Compare
6b06781
to
2d0d8d2
Compare
2d0d8d2
to
a05ba61
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.
The name of the commit Add PParams for Conway
is not accurate I think (I suspect due to rebasing and stuff), so maybe we could change that to mention ConwayGenesis
?
@teodanciu I missed that! 💯 Fixing! |
Also, remove TranslateEra instance for ProposedPPUpdates
a05ba61
to
5896309
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.
Looks great! Thank you!
Description
Add
PParams
toConwayGenesis
forTranslation
.Fixes #2948 and #3160
Checklist
.cabal
andCHANGELOG.md
files according to theversioning process.
.cabal
files for all affected packages are updated. If you change the bounds in a cabal file, that package itself must have a version increase. (See RELEASING.md)CHANGELOG.md
for the affected packages. New section is never added with the code changes. (See RELEASING.md)fourmolu
(usescripts/fourmolize.sh
)scripts/cabal-format.sh
)hie.yaml
has been updated (usescripts/gen-hie.sh
)