Skip to content

Develop -> Master PR#2430

Merged
tynes merged 9 commits intomasterfrom
develop
Apr 6, 2022
Merged

Develop -> Master PR#2430
tynes merged 9 commits intomasterfrom
develop

Conversation

@tynes
Copy link
Contributor

@tynes tynes commented Apr 6, 2022

Description
merge develop to master

smartcontracts and others added 6 commits April 4, 2022 21:37
First commit in a series of commits to clean up the contract tests. Made
a deploy helper function to remove some ugly nested awaits. Also
generally starting to remove unnecessary Factory objects.

Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>
Create a simple script that can set the
config for the  `OVM_GasPriceOracle` using `cast`.
Using the `hardhat` task requires 4gb+
of memory when running.

Also remove the `set-l2-gasprice` hardhat
task that does the same functionality

Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>
* Fix a bunch of nits

chain id enum nit

more utils nits

fix the rest of the nits

fix comment being moved

remove comment that is no longer needed

* add comment to isL2Provider so jsdocs works and change comment

* yarn changeset

Co-authored-by: Will Cory <willcory@Wills-MacBook-Pro.local>
Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>
The labeler wasn't applying `C-Protocol-Critical` labels correctly. I misinterpreted how the `any` rule was supposed to work.
@changeset-bot
Copy link

changeset-bot bot commented Apr 6, 2022

🦋 Changeset detected

Latest commit: 1d2567c

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 6 packages
Name Type
@eth-optimism/go-builder Patch
@eth-optimism/js-builder Patch
@eth-optimism/sdk Minor
@eth-optimism/contracts Patch
@eth-optimism/message-relayer Patch
@eth-optimism/data-transport-layer Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@github-actions github-actions bot added 2-reviewers A-pkg-sdk Area: packages/sdk labels Apr 6, 2022
@mergify mergify bot requested review from cfromknecht and mslipper April 6, 2022 00:46
tynes and others added 3 commits April 6, 2022 15:27
Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>
@tynes tynes merged commit e4fbde2 into master Apr 6, 2022
theochap pushed a commit that referenced this pull request Dec 10, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-pkg-sdk Area: packages/sdk

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants