Skip to content
This repository has been archived by the owner on Dec 2, 2024. It is now read-only.

Bump plutus for important upstream changes #254

Merged
merged 4 commits into from
Jan 18, 2022

Conversation

kk-hainq
Copy link
Contributor

@kk-hainq kk-hainq commented Jan 17, 2022

This PR bumps plutus to:

Most changes are in the third commit that auto-generates test files. Only the first two commits have (small) manual changes. With the second commit addressing this upstream API change: IntersectMBO/plutus#4284.

Pre-submit checklist:

  • Branch
    • Tests are provided (if possible)
    • Commit sequence broadly makes sense
    • Key commits have useful messages
    • Relevant tickets are mentioned in commit messages
    • Formatting, materialized Nix files, PNG optimization, etc. are updated
  • PR
    • Self-reviewed the diff
    • Useful pull request description
    • Reviewer requested

Copy link
Contributor

@raduom raduom left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me, generally, but there are a lot of golden test changes that someone more knowledgeable may want to also review.

@silky
Copy link
Contributor

silky commented Jan 18, 2022

Nice; thanks @kk-hainq ! :)

@silky silky merged commit 34fe6ee into IntersectMBO:main Jan 18, 2022
@kk-hainq kk-hainq deleted the bump-plutus branch January 18, 2022 11:01
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants