Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update EIP-6953: Move to Review #7073

Merged
merged 2 commits into from
Jun 2, 2023
Merged

Update EIP-6953: Move to Review #7073

merged 2 commits into from
Jun 2, 2023

Conversation

timbeiko
Copy link
Contributor

@timbeiko timbeiko commented May 25, 2023

Signed-off-by: Tim Beiko <t.beiko23@gmail.com>
@timbeiko timbeiko requested a review from eth-bot as a code owner May 25, 2023 22:03
@github-actions github-actions bot added c-status Changes a proposal's status s-review This EIP is in Review t-meta labels May 25, 2023
@eth-bot
Copy link
Collaborator

eth-bot commented May 25, 2023

✅ All reviewers have approved.

@eth-bot eth-bot changed the title Change category and add clarification Update EIP-6953: Move to Review May 25, 2023
@eth-bot eth-bot added the e-review Waiting on editor to review label May 25, 2023
status: Draft
type: Informational
status: Review
type: Meta
created: 2023-04-28
requires: 2982, 3675, 6122
Copy link
Contributor Author

Choose a reason for hiding this comment

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

@MariusVanDerWijden can you move 6122 to Review ? thank you!

@github-actions
Copy link

The commit 01d5ec1 (as a parent of 8fbf2c4) contains errors.
Please inspect the Run Summary for details.

EIPS/eip-6953.md Outdated
status: Draft
type: Informational
status: Review
type: Meta
Copy link
Contributor

Choose a reason for hiding this comment

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

I thought we had settled on Informational for this?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

haaa, I wasn't sure where we'd landed. happy to leave it as informational if you think it's best @SamWilsn

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@SamWilsn fixed - should be ready to merge now :-)

@github-actions github-actions bot removed the w-ci Waiting on CI to pass label May 30, 2023
@eth-bot eth-bot enabled auto-merge (squash) June 2, 2023 00:36
Copy link
Collaborator

@eth-bot eth-bot left a comment

Choose a reason for hiding this comment

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

All Reviewers Have Approved; Performing Automatic Merge...

@eth-bot eth-bot merged commit e5f4c0e into ethereum:master Jun 2, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
c-status Changes a proposal's status e-review Waiting on editor to review s-review This EIP is in Review t-informational
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants