-
Notifications
You must be signed in to change notification settings - Fork 720
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 chaps & integration #5243
Conversation
893a6cb
to
9cb1fa2
Compare
127b522
to
6436dd5
Compare
c35b7b9
to
010ac36
Compare
010ac36
to
f9241ba
Compare
3d13e4d
to
298178b
Compare
We'll need to merge #5275. |
c56d9c7
to
0fcf262
Compare
e073a02
to
56e157a
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.
LGTM
cardano-cli/CHANGELOG.md
Outdated
@@ -2,7 +2,8 @@ | |||
|
|||
## 8.1.0 | |||
|
|||
- | |||
- Updated ledger dependency bounds | |||
- Support for query flag in `Ping` parser |
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.
Where can I find more documentation regarding this?
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.
Perhaps @coot or @jprider63 knows?
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.
Here is the corresponding issue and initial PR.
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.
Thanks. Can you update the CHANGELOG following the format of the other entries? @jprider63 your team should provide something to @JaredCorduan that he can simply copy paste.
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.
ok, I changed them, do they look okay now?
d174cb3
to
f84df59
Compare
* Change ledger bounds in cardano-api, cardano-cli. * Add Plutus V3 support to cardano-api, only available in the conway ledger era. * PState is now parametric in era, not crypto. * Adjust for new conway era certificates. Note that the certificate design is currently wrong in ledger, making the conway support in cardano-api type-checking but broken. * Ledger UMapCompact is now UMap * Ledger types with names involving DState are renamed to CertState. * Add the new query flag to the CLI ping.
f84df59
to
6328af9
Compare
in | ||
'' | ||
export CARDANO_NODE_SRC=${filteredProjectBase} | ||
''; |
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.
CARDANO_NODE_SRC
fix
Description
I have updated CHaPs and done the following integration work (with help from others!):
cardano-api
,cardano-cli
.cardano-api
, only available in the conway ledger era.PState
is now parametric in era, not crypto.Note that the certificate design is currently wrong in ledger,
making the conway support in
cardano-api
type-checking but broken.UMapCompact
is nowUMap
DState
are renamed toCertState
.Checklist
See Runnings tests for more details
CHANGELOG.md
for affected package.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
Note on CI
If your PR is from a fork, the necessary CI jobs won't trigger automatically for security reasons.
You will need to get someone with write privileges. Please contact IOG node developers to do this
for you.