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

fix: use owner as CREATE2 salt #123

Merged
merged 1 commit into from
Jun 28, 2023

Conversation

PaulRBerg
Copy link
Owner

@PaulRBerg PaulRBerg requested a review from andreivladbrg June 18, 2023 12:46
@PaulRBerg PaulRBerg force-pushed the fix/plugin-method-collision branch 2 times, most recently from fadb724 to 43b7b36 Compare June 19, 2023 11:55
@PaulRBerg PaulRBerg force-pushed the fix/plugin-method-collision branch 2 times, most recently from a5a11ad to ece321d Compare June 19, 2023 12:12
src/PRBProxyRegistry.sol Outdated Show resolved Hide resolved
src/PRBProxyRegistry.sol Outdated Show resolved Hide resolved
@PaulRBerg PaulRBerg force-pushed the fix/plugin-method-collision branch 2 times, most recently from 5730bf9 to 9ad9ae0 Compare June 22, 2023 10:35
@PaulRBerg PaulRBerg closed this Jun 22, 2023
@PaulRBerg PaulRBerg reopened this Jun 22, 2023
@PaulRBerg PaulRBerg force-pushed the fix/plugin-method-collision branch from 9ad9ae0 to 6c92d17 Compare June 24, 2023 09:09
docs: improve writing in README
refactor: delete "nextSeeds"
refactor: delete stale CREATE2 seeds
refactor: remove "origin" in "DeployProxy" event
test: remove unused constants
test: update tests in light of new contract logic
test: remove stale "origin" fuzzing
@PaulRBerg PaulRBerg merged commit 25f7c8b into fix/plugin-method-collision Jun 28, 2023
@PaulRBerg PaulRBerg deleted the fix/create2-salt branch June 28, 2023 18:51
@PaulRBerg PaulRBerg restored the fix/create2-salt branch June 29, 2023 13:37
@PaulRBerg
Copy link
Owner Author

Yanked in favor of #130.

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.

3 participants