-
Notifications
You must be signed in to change notification settings - Fork 23
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
DRep extended key: add CastVerificationKeyRole #327
Conversation
$ vk | ||
where | ||
impossible = | ||
error "castVerificationKey: byron and shelley key sizes do not match!" |
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 get that rawDeserialiseVerKeyDSIGN
fails if psbFromByteStringCheck
(which is a length check) fails. But the eras I don't understand.
Maybe the message should also say something about extended vs. non-extended DRep keys?
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 error message is definitely a copy-paste. You will need to go code spelunking to figure out a sensible error message.
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.
Indeed it's a copy-paste, which I assumed safe since all other instances of CastVerificationKeyRole
do the same.
@carlhammann> I did disambiguate the error message to mention DRep
though. Could make our life easier to debug in the future 👍
Since you approved @Jimbo4350, I'm assuming this is good to go, and so I'm not digging into more details.
1bc2112
to
d133e42
Compare
…-are-not-golden-files-to-input-directory Move files that are not golden files into `input` directory
Changelog
Context
Required for IntersectMBO/cardano-cli#377. Should have been part of #320 but somehow I lost this part during my local rebases 🙃
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