Skip to content

bedrock: Proxy Admin improvements#2787

Merged
mergify[bot] merged 3 commits intodevelopfrom
m/polish-proxy
Jun 15, 2022
Merged

bedrock: Proxy Admin improvements#2787
mergify[bot] merged 3 commits intodevelopfrom
m/polish-proxy

Conversation

@maurelian
Copy link
Copy Markdown
Contributor

@maurelian maurelian commented Jun 15, 2022

Description
Some changes to improve the Proxy Admin. See inline comments.

@changeset-bot
Copy link
Copy Markdown

changeset-bot bot commented Jun 15, 2022

🦋 Changeset detected

Latest commit: 7897ba2

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

This PR includes changesets to release 1 package
Name Type
@eth-optimism/contracts-bedrock 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

@tynes
Copy link
Copy Markdown
Contributor

tynes commented Jun 15, 2022

Need to regenerate the gas snapshot

@maurelian maurelian force-pushed the m/polish-proxy branch 2 times, most recently from 3929568 to 093c05f Compare June 15, 2022 17:51
@tynes
Copy link
Copy Markdown
Contributor

tynes commented Jun 15, 2022

This could also use a changeset

@mergify
Copy link
Copy Markdown
Contributor

mergify bot commented Jun 15, 2022

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

@mergify mergify bot merged commit f23bae0 into develop Jun 15, 2022
@mergify mergify bot deleted the m/polish-proxy branch June 15, 2022 18:47
@mergify
Copy link
Copy Markdown
Contributor

mergify bot commented Jun 15, 2022

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

@mergify mergify bot removed the on-merge-train label Jun 15, 2022
@mslipper mslipper mentioned this pull request Jun 18, 2022
theochap added a commit that referenced this pull request Dec 10, 2025
## Description

Simple PR that improves the connection drop tests by ensuring that the
unsafe head doesn't advance before consolidation
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