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-7401: Move to Review #7414

Merged
merged 4 commits into from
Aug 22, 2023

Conversation

ThunderDeliverer
Copy link
Contributor

This PR updates the status of EIP-7401 to Review stage.

@github-actions github-actions bot added c-status Changes a proposal's status s-review This EIP is in Review t-erc labels Jul 26, 2023
@eth-bot
Copy link
Collaborator

eth-bot commented Jul 26, 2023

✅ All reviewers have approved.

@eth-bot eth-bot added the e-review Waiting on editor to review label Jul 26, 2023
The proposal's number was updated to reflect it being a fix of ERC-6059.
@github-actions github-actions bot removed c-status Changes a proposal's status t-erc s-review This EIP is in Review labels Aug 8, 2023
@eth-bot eth-bot added the e-consensus Waiting on editor consensus label Aug 8, 2023
EIPS/eip-7059.md Outdated Show resolved Hide resolved
@github-actions github-actions bot added c-status Changes a proposal's status s-review This EIP is in Review t-erc labels Aug 9, 2023
Comment on lines +14 to +19
> :information_source: **[ERC-7401](./eip-7401.md) has superseded [ERC-6059](./eip-6059.md).** :information_source:
> [ERC-7401](./eip-7401.md) fixes the inconsistency in the [ERC-6059](./eip-6059.md) interface specification, where interface ID doesn't match the interface specified as the interface evolved during the proposal's lifecycle, but one of the parameters was not added to it. The missing parameter is, however, present in the interface ID.
> Apart from this fix, [ERC-7401](./eip-7401.md) is functionally equivalent to [ERC-6059](./eip-6059.md).
>
> :warning: [ERC-7401](./eip-7401.md) MUST be used in lieu of [ERC-6059](./eip-6059.md). :warning:

Copy link
Contributor

Choose a reason for hiding this comment

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

:information_source: doesn't actually render to anything. I guess use an emoji?

I'd prefer this to be moved into the motivation section, but I won't be too strict about that.

@eth-bot eth-bot enabled auto-merge (squash) August 22, 2023 15:07
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 c139c47 into ethereum:master Aug 22, 2023
11 checks passed
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-consensus Waiting on editor consensus e-review Waiting on editor to review s-review This EIP is in Review t-erc
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants