Skip to content

fix(ctb): remove unnecessary L2OO init param#3675

Merged
mergify[bot] merged 4 commits intodevelopfrom
sc/ctb-oo-fix-init
Oct 10, 2022
Merged

fix(ctb): remove unnecessary L2OO init param#3675
mergify[bot] merged 4 commits intodevelopfrom
sc/ctb-oo-fix-init

Conversation

@smartcontracts
Copy link
Contributor

Description
Removes an unnecessary parameter from the L200 initialization function since this parameter is already set in the constructor.

@changeset-bot
Copy link

changeset-bot bot commented Oct 8, 2022

🦋 Changeset detected

Latest commit: 1636250

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

This PR includes changesets to release 5 packages
Name Type
@eth-optimism/contracts-bedrock Patch
@eth-optimism/actor-tests Patch
@eth-optimism/sdk Patch
@eth-optimism/drippie-mon Patch
@eth-optimism/message-relayer 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-contracts-bedrock Area: packages/contracts-bedrock labels Oct 8, 2022
@mergify
Copy link
Contributor

mergify bot commented Oct 8, 2022

This PR changes implementation code, but doesn't include a changeset. Did you forget to add one?

Removes an unnecessary parameter from the L200 initialization function
since this parameter is already set in the constructor.
@tynes tynes requested review from maurelian and tynes October 8, 2022 23:04
@mergify
Copy link
Contributor

mergify bot commented Oct 10, 2022

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

@mergify
Copy link
Contributor

mergify bot commented Oct 10, 2022

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

@mergify mergify bot merged commit f951f65 into develop Oct 10, 2022
@mergify mergify bot deleted the sc/ctb-oo-fix-init branch October 10, 2022 03:05
@mergify mergify bot removed the on-merge-train label Oct 10, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-pkg-contracts-bedrock Area: packages/contracts-bedrock

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants