Skip to content

Conversation

grantmike
Copy link
Contributor

No description provided.

@grantmike grantmike merged commit 6e7513c into circlefin:master Mar 11, 2025
4 checks passed
zzhang-circle pushed a commit to zzhang-circle/evm-cctp-contracts that referenced this pull request Jun 20, 2025
… CREATE2 more effectively (circlefin#60)

### Summary 

Follow-up to:
circlefin/evm-cctp-contracts-private@8cf3673,
this correct an oversight where the TokenMinterV2 via Create2 was being
passed the tokenController as a constructor arg. Since this affects the
bytecode, it affects the address.

### Testing

The tests actually already assert that the tokenController is rotated
correctly; it's done atomically in both ways, only this way ensures a
consistent address.

Edit: added a test to ensure that the script is deploying via CREATE2 in
an expected way.
zzhang-circle pushed a commit to zzhang-circle/evm-cctp-contracts that referenced this pull request Jun 20, 2025
… CREATE2 more effectively (circlefin#60)

### Summary 

Follow-up to:
circlefin/evm-cctp-contracts-private@8cf3673,
this correct an oversight where the TokenMinterV2 via Create2 was being
passed the tokenController as a constructor arg. Since this affects the
bytecode, it affects the address.

### Testing

The tests actually already assert that the tokenController is rotated
correctly; it's done atomically in both ways, only this way ensures a
consistent address.

Edit: added a test to ensure that the script is deploying via CREATE2 in
an expected way.
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