Skip to content

Update EIP-1: add devp2p repo as accepted external resource#6034

Merged
SamWilsn merged 2 commits intoethereum:masterfrom
lightclient:add-devp2p-ext-resource
Nov 28, 2022
Merged

Update EIP-1: add devp2p repo as accepted external resource#6034
SamWilsn merged 2 commits intoethereum:masterfrom
lightclient:add-devp2p-ext-resource

Conversation

@lightclient
Copy link
Copy Markdown
Member

This add the Ethereum networking specs (devp2p) as an accepted resource to EIP-1.

@lightclient lightclient requested a review from eth-bot as a code owner November 22, 2022 21:48
@github-actions github-actions bot added c-update Modifies an existing proposal t-process labels Nov 22, 2022
@eth-bot
Copy link
Copy Markdown
Collaborator

eth-bot commented Nov 23, 2022

Hi! I'm a bot, and I wanted to automerge your PR, but couldn't because of the following issue(s):


(fail) eip-1.md

classification
updateEIP
  • Changes to EIP 1 require at least 5 unique approvals from editors; there's currently 1 approvals; the remaining editors are @axic, @SamWilsn, @Pandapip1, @gcolvin

@Pandapip1 Pandapip1 changed the title EIP-1: add devp2p repo as accepted external resource Update EIP-1: add devp2p repo as accepted external resource Nov 23, 2022

### Networking Specifications

Links to the Ethereum Networking Specifications may be included using normal markdown syntax, such as:
Copy link
Copy Markdown
Member

@Pandapip1 Pandapip1 Nov 23, 2022

Choose a reason for hiding this comment

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

Suggested change
Links to the Ethereum Networking Specifications may be included using normal markdown syntax, such as:
Links to specific commits and files in specific commits of the Ethereum Networking Specifications GitHub repository may be included using the normal markdown link syntax, such as:

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

The text should be consistent between consensus and networking specs.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Can I suggest that the text be changed for both, then?

@SamWilsn
Copy link
Copy Markdown
Contributor

I'll make @Pandapip1's changes in another PR.

@SamWilsn SamWilsn merged commit 66a9fc3 into ethereum:master Nov 28, 2022
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 t-process

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants