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

feat: add ERC-5202 Blueprint library with initial tests #11700

Merged
merged 1 commit into from
Sep 3, 2024

Conversation

mds1
Copy link
Contributor

@mds1 mds1 commented Aug 30, 2024

As suggested in #11623 (comment), to help shrink the size of that PR we pull out the Blueprint library into it's own code. This pattern is used so OP Stack Manager can deploy various contracts without bloating its code size.

The tests added are light, and additional testing of this library will be added soon

@mds1
Copy link
Contributor Author

mds1 commented Aug 30, 2024

cc @charles-cooper in case you have any thoughts, I haven't seen a solidity implementation anywhere yet :)

@tynes
Copy link
Contributor

tynes commented Aug 31, 2024

cc @Vectorized Have you considered ERC-5202 for solady?

@tynes tynes added this pull request to the merge queue Sep 3, 2024
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Sep 3, 2024
@tynes tynes added this pull request to the merge queue Sep 3, 2024
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Sep 3, 2024
@mds1 mds1 added this pull request to the merge queue Sep 3, 2024
Merged via the queue into develop with commit 957e13d Sep 3, 2024
57 checks passed
@mds1 mds1 deleted the feat/blueprints branch September 3, 2024 21:52
@mds1 mds1 mentioned this pull request Sep 6, 2024
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