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-1820: Fix same tagging issue #7398

Merged
merged 1 commit into from
Aug 20, 2023
Merged

Conversation

Pandapip1
Copy link
Member

See also #7397

@Pandapip1 Pandapip1 requested a review from eth-bot as a code owner July 25, 2023 15:53
@Pandapip1 Pandapip1 added this to the Manual Merge Queue milestone Jul 25, 2023
@github-actions github-actions bot added c-update Modifies an existing proposal s-final This EIP is Final t-erc labels Jul 25, 2023
@eth-bot
Copy link
Collaborator

eth-bot commented Jul 25, 2023

File EIPS/eip-1820.md

Requires 2 more reviewers from @axic, @gcolvin, @lightclient, @SamWilsn

@eth-bot eth-bot added the e-consensus Waiting on editor consensus label Jul 25, 2023
@github-actions
Copy link

The commit fb32854 (as a parent of 3ddf4bc) contains errors.
Please inspect the Run Summary for details.

@github-actions github-actions bot added the w-ci Waiting on CI to pass label Jul 25, 2023
@@ -339,7 +339,9 @@ The contract has the address above for every chain on which it is deployed.
<details>
<summary>Raw metadata of <code>./contracts/ERC1820Registry.sol</code></summary>
<pre>
Copy link
Contributor

Choose a reason for hiding this comment

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

Still has an extra <pre>, no?

Copy link
Member Author

Choose a reason for hiding this comment

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

Don't think so: L559/L663

@SamWilsn SamWilsn merged commit eed0152 into master Aug 20, 2023
@SamWilsn SamWilsn deleted the Pandapip1-patch-13 branch August 20, 2023 14:46
streamnft-tech pushed a commit to streamnft-tech/EIPs that referenced this pull request Oct 27, 2023
RaphaelHardFork pushed a commit to RaphaelHardFork/EIPs that referenced this pull request Jan 30, 2024
just-a-node pushed a commit to connext/EIPs that referenced this pull request Feb 17, 2024
GAEAlimited pushed a commit to GAEAlimited/EIPs that referenced this pull request Jun 19, 2024
blacksnow2 pushed a commit to blacksnow2/EIPs that referenced this pull request Jul 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
c-update Modifies an existing proposal e-consensus Waiting on editor consensus s-final This EIP is Final t-erc w-ci Waiting on CI to pass
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants