Skip to content

feat(core): use ethers submodules#3363

Merged
mergify[bot] merged 2 commits intodevelopfrom
sc/core-ethers-subs
Sep 9, 2022
Merged

feat(core): use ethers submodules#3363
mergify[bot] merged 2 commits intodevelopfrom
sc/core-ethers-subs

Conversation

@smartcontracts
Copy link
Contributor

Description
MetaMask requested that we use ethers submodules instead of importing the entire ethers package. Relatively straightforward change.

Tests
No test changes are required, no logic is changing.

@changeset-bot
Copy link

changeset-bot bot commented Sep 8, 2022

🦋 Changeset detected

Latest commit: 749c2d4

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

This PR includes changesets to release 12 packages
Name Type
@eth-optimism/core-utils Minor
@eth-optimism/actor-tests Patch
@eth-optimism/common-ts Patch
@eth-optimism/contracts-bedrock Patch
@eth-optimism/contracts-governance Patch
@eth-optimism/contracts Patch
@eth-optimism/data-transport-layer Patch
@eth-optimism/drippie-mon Patch
@eth-optimism/fault-detector Patch
@eth-optimism/message-relayer Patch
@eth-optimism/replica-healthcheck Patch
@eth-optimism/sdk 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 the A-pkg-core-utils Area: packages/core-utils label Sep 8, 2022
MetaMask requested that we use ethers submodules instead of importing
the entire ethers package. Relatively straightforward change.
@tynes tynes requested a review from maurelian September 9, 2022 16:10
Copy link

@adonesky1 adonesky1 left a comment

Choose a reason for hiding this comment

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

LGTM!

@mergify
Copy link
Contributor

mergify bot commented Sep 9, 2022

This PR has been added to the merge queue, and will be merged soon.

@mergify
Copy link
Contributor

mergify bot commented Sep 9, 2022

This PR is next in line to be merged, and will be merged as soon as checks pass.

@mergify mergify bot merged commit dbfea11 into develop Sep 9, 2022
@mergify mergify bot deleted the sc/core-ethers-subs branch September 9, 2022 18:34
@mergify mergify bot removed the on-merge-train label Sep 9, 2022
maurelian pushed a commit that referenced this pull request Sep 15, 2022
MetaMask requested that we use ethers submodules instead of importing
the entire ethers package. Relatively straightforward change.

Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>
sam-goldman pushed a commit that referenced this pull request Sep 15, 2022
MetaMask requested that we use ethers submodules instead of importing
the entire ethers package. Relatively straightforward change.

Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-pkg-core-utils Area: packages/core-utils

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants