Skip to content
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

CIP-0057: ContractBlueprint with derived schema definitions and safe schema refs #5824

Merged
merged 5 commits into from
Mar 18, 2024

Conversation

Unisay
Copy link
Contributor

@Unisay Unisay commented Mar 8, 2024

The idea is that user specifies at most 3 types: Param, Datum, Redeemer and all the types nested withing them are extracted automatically (un-rolled).

@Unisay Unisay requested review from michaelpj and zliu41 March 8, 2024 17:07
@Unisay Unisay force-pushed the yura/PLT-9657/derive-schema-types branch from 5ad7a0e to b2bcbf9 Compare March 11, 2024 10:42
@Unisay Unisay changed the title CIP-57: unroll top level types CIP-0057: unroll top level types Mar 11, 2024
@Unisay Unisay self-assigned this Mar 11, 2024
@Unisay Unisay added the No Changelog Required Add this to skip the Changelog Check label Mar 11, 2024
@michaelpj
Copy link
Contributor

I feel like I've lost the context again. Can we have a Note explaining what the deal is with nested types inside a schema and why we need to handle them in a special way?

@Unisay Unisay force-pushed the yura/PLT-9657/derive-schema-types branch 2 times, most recently from 4605a2d to 961d455 Compare March 11, 2024 15:47
@Unisay Unisay force-pushed the yura/PLT-9657/derive-schema-types branch from 961d455 to c3b57a3 Compare March 11, 2024 15:50
Unisay and others added 4 commits March 18, 2024 14:43
…initons. (#5839)

* Add pointer to issue on the ledger's cost model interface (#5828)

* Agda html (#5827)

* Moved all Agda files to Markdown.

* Added some brief text to the TOC

* Fixed the file extension in Setup.hs for the Markdown Agda.

* Added all possible file extensions and the preprocessor seems to run now.

* Cleaned out .lagda from the cabal file, since everything is .lagda.md now.

* chore(deps): bump tj-actions/changed-files from 42 to 43 (#5835)

Bumps [tj-actions/changed-files](https://github.com/tj-actions/changed-files) from 42 to 43.
- [Release notes](https://github.com/tj-actions/changed-files/releases)
- [Changelog](https://github.com/tj-actions/changed-files/blob/main/HISTORY.md)
- [Commits](tj-actions/changed-files@v42...v43)

---
updated-dependencies:
- dependency-name: tj-actions/changed-files
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Kwxm/correct conformance comments (#5829)

* Fix comments in conformance tests

* Fix comments in conformance tests

* Fix comments in conformance tests

* CIP-57: unroll top level types

* Extract `goldenJson` function and move it to the `Tests.Lib`

* Note about unrolling types

* CIP-57: accumulate schema definitions preserving types for safe refs.

* Build schema definitions by un-rolling a list of top-level types.

* validatorParameters :: []

---------

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: Kenneth MacKenzie <[email protected]>
Co-authored-by: Ramsay Taylor <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
@Unisay Unisay force-pushed the yura/PLT-9657/derive-schema-types branch from c3b3cea to 210aa9e Compare March 18, 2024 13:43
* Derive validator blueprints, extend blueprint test to use 2 validators.

* Compile typed validator

* Example with 2 parameters

* fail fast in case of duplicate annotations
@Unisay Unisay force-pushed the yura/PLT-9657/derive-schema-types branch from 210aa9e to 7755732 Compare March 18, 2024 13:47
@Unisay Unisay changed the title CIP-0057: unroll top level types CIP-0057: ContractBlueprint with derived schema definitions and safe schema refs Mar 18, 2024
@Unisay Unisay enabled auto-merge (squash) March 18, 2024 13:51
@Unisay
Copy link
Contributor Author

Unisay commented Mar 18, 2024

I am going to merge this PR as it contains pieces that were already reviewed in other PRs and merged here. I believe that I have addressed all the expressed concerns. If any concerns remain, though, I'd be happy to address them in next PRs.

@Unisay Unisay merged commit ac41901 into master Mar 18, 2024
5 checks passed
@Unisay Unisay deleted the yura/PLT-9657/derive-schema-types branch March 18, 2024 15:02
v0d1ch pushed a commit to v0d1ch/plutus that referenced this pull request Dec 6, 2024
…schema refs (IntersectMBO#5824)

CIP-0057: ContractBlueprint with derived schema definitions and safe schema refs (IntersectMBO#5824)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
No Changelog Required Add this to skip the Changelog Check
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants