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 superchain erc20 baseline #37

Merged

Conversation

agusduha
Copy link
Member

@agusduha agusduha commented Aug 29, 2024

Closes OPT-200

@agusduha agusduha requested a review from 0xng August 29, 2024 14:51
@agusduha agusduha self-assigned this Aug 29, 2024
Copy link

linear bot commented Aug 29, 2024

@defi-wonderland defi-wonderland deleted a comment from linear bot Aug 29, 2024
}

/// @notice Test that the contract's `initializer` sets the correct values.
function test_initializer_succeeds() public view {
Copy link
Member

Choose a reason for hiding this comment

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

should this be constructor instead of initializer or did you use the initializer word due to the unstructured storage?

@agusduha agusduha merged commit 0d0245b into sc/superchain-erc20-baseline Aug 29, 2024
3 checks passed
@agusduha agusduha deleted the feat/add-superchain-erc20-baseline branch August 29, 2024 21:07
0xng pushed a commit that referenced this pull request Sep 11, 2024
* feat: add superchain erc20 baseline

* feat: make superchain ERC20 simpler

* fix: small version fix and tests

* test: fix test name
0xng pushed a commit that referenced this pull request Sep 12, 2024
* feat: add superchain erc20 baseline

* feat: make superchain ERC20 simpler

* fix: small version fix and tests

* test: fix test name
0xteddybear pushed a commit that referenced this pull request Sep 17, 2024
* feat: add superchain erc20 baseline (#37)

* feat: add superchain erc20 baseline

* feat: make superchain ERC20 simpler

* fix: small version fix and tests

* test: fix test name

* test: remove unused import

* feat: making baseline abstract

* fix: interfaces to comply with the new interface checker

* fix: import paths and empty line

* fix: lint line

---------

Co-authored-by: 0xng <[email protected]>
Co-authored-by: 0xng <[email protected]>
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