Skip to content

Add Utils for Full Node Provider#2

Merged
K-Ho merged 1 commit intomasterfrom
YAS153/OpenSourceOVM/Tutorial
Feb 12, 2020
Merged

Add Utils for Full Node Provider#2
K-Ho merged 1 commit intomasterfrom
YAS153/OpenSourceOVM/Tutorial

Conversation

@K-Ho
Copy link
Copy Markdown
Contributor

@K-Ho K-Ho commented Feb 11, 2020

Description

Add DeployContract, CreateMockProvider, GetWallets

Contributing Agreement

@K-Ho K-Ho merged commit db6418e into master Feb 12, 2020
@willmeister willmeister deleted the YAS153/OpenSourceOVM/Tutorial branch February 14, 2020 23:44
snario pushed a commit that referenced this pull request Apr 14, 2021
Ports over most of the tests from Contracts V1
smartcontracts added a commit that referenced this pull request Nov 22, 2022
Additional tweaks to prevent test flakes in the SDK.
mergify bot added a commit that referenced this pull request Nov 22, 2022
dome pushed a commit to jfinchain/optimism that referenced this pull request Apr 7, 2023
refcell pushed a commit that referenced this pull request Aug 14, 2023
refcell pushed a commit that referenced this pull request Sep 6, 2023
Gas Optimization: Cache resolver.isPayable() external call
@coderabbitai coderabbitai bot mentioned this pull request Dec 21, 2023
nitaliano pushed a commit that referenced this pull request May 20, 2024
Additional tweaks to prevent test flakes in the SDK.
shyam-patel-kira added a commit to shyam-patel-kira/optimism that referenced this pull request May 20, 2025
blockchaindevsh added a commit to blockchaindevsh/optimism that referenced this pull request Jun 30, 2025
smartcontracts added a commit that referenced this pull request Feb 22, 2026
…16

Add missing @param blueprint NatSpec to OpcmContractRef struct (#2).
Add comments about pause blocking interop upgrades (#3). Document
migrate() scope limitations and re-migration risks (#7, #15). Update
PERMIT_ALL_CONTRACTS_INSTRUCTION comment (#12). Document intentional
use of chainSystemConfigs[0] for shared contracts (#16).

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
github-merge-queue bot pushed a commit that referenced this pull request Mar 3, 2026
…16 (#19271)

Add missing @param blueprint NatSpec to OpcmContractRef struct (#2).
Add comments about pause blocking interop upgrades (#3). Document
migrate() scope limitations and re-migration risks (#7, #15). Update
PERMIT_ALL_CONTRACTS_INSTRUCTION comment (#12). Document intentional
use of chainSystemConfigs[0] for shared contracts (#16).

Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
OptimismBot pushed a commit that referenced this pull request Mar 3, 2026
smartcontracts added a commit that referenced this pull request Mar 6, 2026
The 5s timeout is too tight under CI load. When 12 parallel test nodes
compete for CPU/IO on a 2xlarge box, Anvil sometimes takes >5s to print
its "Listening on" line, triggering "anvil did not start in time" in
TestImplementations and TestSuperchain. This is the #2 and #3 most
frequent flake in the repo over the last 7 days (67 and 46 incidences).

30s gives Anvil enough headroom on a loaded machine while still failing
fast on a genuine startup failure.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
github-merge-queue bot pushed a commit that referenced this pull request Mar 6, 2026
The 5s timeout is too tight under CI load. When 12 parallel test nodes
compete for CPU/IO on a 2xlarge box, Anvil sometimes takes >5s to print
its "Listening on" line, triggering "anvil did not start in time" in
TestImplementations and TestSuperchain. This is the #2 and #3 most
frequent flake in the repo over the last 7 days (67 and 46 incidences).

30s gives Anvil enough headroom on a loaded machine while still failing
fast on a genuine startup failure.

Co-authored-by: smartcontracts <smartcontracts@users.noreply.github.com>
Co-authored-by: Claude Sonnet 4.6 <noreply@anthropic.com>
github-merge-queue bot pushed a commit that referenced this pull request Mar 9, 2026
* feat: add nut lib

* refactor: remove newTx function from nut lib

* feat: add GenerateNUTBundle script

* fix: pre-pr fixes

* test: add generate bundle utils tests

* refactor: replace custom computeCreate2Address by forge's

* docs: fix and add natspec

* fix: remove script inheritance from utils contract

* refactor: remove unnecesary fields from nut tx struct & replace sourcehash by intent

* test: add create tx empry args equivalence and different salts assumption

* refactor: remove fork, salt and cgt from bundle script input & consolidate utils into UpgradeUtils

* refactor: remove fork, salt and cgt from bundle script input

* refactor: comments and constants

* refactor: consolidate nut bundle utilities into UpgradeUtils

* fix: proxy admin get code

* fix: remove unused import Fork

* fix: rm L2ContractsManagerTypes

* fix: update impl struct and make lib functions internal

* feat: remove TODO for L2CM merge and add TODO for OptimismMintableERC721Factory initializable upgrade

* test: complete nuts structure testing

* feat: add metadata struct to bundle

* feat: make OptimismMintableERC721Factory initializable

* fix: replace jovian for karst

* fix: add Initialized event to IOptimismMintableERC721Factory

* fix: remove check immutable in apply tests

* fix: outdated safety invariants

* fix: add OptimismMintableERC721FactoryLegacyMapping

* refactor: rm upgrade name from intent, add upgradeBundlePath helper and add TODO for karst nuts removal

* feat: add invariant #2 to L2ProxyAdmin

* feat: add nut bundle check, just script and ci check

* feat: add implementation count check

* docs: add array string clarification comment

* feat: add regex check for metadata version and 'to' field comment

* fix: remove salt standard todo comment

* fix: remove old upgrade OptimismMintableERC721Factory call and fix initializable slot

* feat: add IOptimismMintableERC721FactoryLegacyMapping

* fix: run pre-pr

* fix: run pre-pr

* fix: remove owner argument for L2PA constructor and set owner to zero address

* fix: remove gitkeep

* fix: storage gap in OptimismMintableERC721Factory

* refactor: make bundle file name generic

* fix: remove duplicated assert

* fix: l2genesis proxy admin impl owner test

* fix: l2pa createAdmin with manual storage setting test
wwared pushed a commit that referenced this pull request Mar 9, 2026
The 5s timeout is too tight under CI load. When 12 parallel test nodes
compete for CPU/IO on a 2xlarge box, Anvil sometimes takes >5s to print
its "Listening on" line, triggering "anvil did not start in time" in
TestImplementations and TestSuperchain. This is the #2 and #3 most
frequent flake in the repo over the last 7 days (67 and 46 incidences).

30s gives Anvil enough headroom on a loaded machine while still failing
fast on a genuine startup failure.

Co-authored-by: smartcontracts <smartcontracts@users.noreply.github.com>
Co-authored-by: Claude Sonnet 4.6 <noreply@anthropic.com>
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.

1 participant