-
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
Avoid using sequence of tuples, by adding GovActionId to GovActionState #3637
Conversation
2059b49
to
41a4f60
Compare
41a4f60
to
75626d3
Compare
8d7762e
to
90fc897
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.
Very nice. Found some remnants of the previous Entry type, but otherwise it looks great!
Thank you.
libs/cardano-ledger-test/src/Test/Cardano/Ledger/Examples/ConwayFeatures.hs
Outdated
Show resolved
Hide resolved
libs/cardano-ledger-test/src/Test/Cardano/Ledger/Examples/ConwayFeatures.hs
Outdated
Show resolved
Hide resolved
4fa9fcf
to
69a4a52
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 good to me (except you left an import of the inexistent insertGovActionEntry,
in Governance
)
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!
69a4a52
to
d119db1
Compare
to reflect breaking changes
45bc2a5
to
a79e78d
Compare
|
||
## 1.7.0.0 | ||
|
||
* Add `Network` validation for `ProposalProcedure` and `TreasuryWithdrawals` in GOV #3659 |
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.
Did this line just get removed? Should it be part of 1.8
instead? @Soupstraw
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.
Oops, looks like I removed it by mistake. Yes, I think it should be part of 1.8 because 1.7 was released at (bbdfc3f) before this commit . Sorry!
Description
closes #3578
This PR fixes the above ticket in a different way. Namely, by adding
govActionId
to theGovActionState
. This approach is very similar to howPoolParams
are implemented.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
)