-
Notifications
You must be signed in to change notification settings - Fork 202
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
leger-api: add GenMap support #3506
Conversation
Should I add a line in the release notes ? |
...ledger-api-common/src/main/scala/com/digitalasset/ledger/api/validation/ValueValidator.scala
Show resolved
Hide resolved
IMO it's best to leave |
I would like, but this change will break scala/haskell code (with a non exhaustive pattern matching) that relies on the code generated from the proto definition. |
2c00c45
to
8372e30
Compare
8ce247f
to
6986f67
Compare
* leger-api: add GenMap support * cosmetic * Address Gerolf's comment * add a point in the release notes * Address Fran's comments
This PR advances #2256.
It adds support of GenMap to the ledger API.
Pull Request Checklist
NOTE: CI is not automatically run on non-members pull-requests for security
reasons. The reviewer will have to comment with
/AzurePipelines run
totrigger the build.