-
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
Update NewCommittee
action to use Credential and add more info
#3557
Conversation
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 should have said this in the ticket, but it was only after I looked at the code I realized that it would be good to add a whole new type for the Committee
eras/conway/impl/src/Cardano/Ledger/Conway/Governance/Procedures.hs
Outdated
Show resolved
Hide resolved
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 FYI, there is WIP to make the CommitteeColdKey
from a KeyHash
to a Credential
#3564
eb2a2bf
to
4863bdf
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.
👍
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 a single doubt (question), but otherwise looks good! 🙌
eras/conway/impl/src/Cardano/Ledger/Conway/Governance/Procedures.hs
Outdated
Show resolved
Hide resolved
…ittee`: namely: old committee and the expiration epoch for each new member. Also update `EnactState` to reflect these changes
4863bdf
to
44d65b3
Compare
namely: old committee and the expiration epoch for each new member. Also update
EnactState
to reflect these changesCloses #3543
Draft because I haven't figured out how to update scriptsNeeded for this
Description
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
)