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

Support Plutus v2 #110

Closed
wants to merge 3 commits into from
Closed

Support Plutus v2 #110

wants to merge 3 commits into from

Conversation

kk-hainq
Copy link
Contributor

@kk-hainq kk-hainq commented Nov 15, 2021

This is more or less a proof-of-concept thing, I wonder what is the actual migration plan at the moment?
Also, note that the code changes in the first two commits are very minimal. The huge diff is mostly on affected tests, probably due to the change in ScriptContext's structure.

Key Plutus v2 PRs:

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

@kk-hainq kk-hainq marked this pull request as draft November 15, 2021 01:22
@silky
Copy link
Contributor

silky commented Dec 31, 2021

I'm going to close this @kk-hainq , as I think Plutus V2 is not on the near horizon; but feel free to re-open as you see fit!

@silky silky closed this Dec 31, 2021
@kk-hainq kk-hainq deleted the plutus-v2 branch December 31, 2021 11:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants