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-template: Add chain-specific considerations #9457

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Marchhill
Copy link
Contributor

Added a section to the EIP template to address any chain-specific considerations.

The aim is to reduce the likelihood of incidents like the recent one on Sepolia, which are caused by minor differences between chains. If EIP authors address these considerations then such problems could be caught much earlier.

@Marchhill Marchhill requested a review from eth-bot as a code owner March 5, 2025 17:38
@github-actions github-actions bot added c-update Modifies an existing proposal s-draft This EIP is a Draft t-process labels Mar 5, 2025
@eth-bot
Copy link
Collaborator

eth-bot commented Mar 5, 2025

🛑 eip-review-bot failed for an unknown reason. Please see logs for more details, and report this issue at the eip-review-bot repository.

@eth-bot eth-bot added the e-consensus Waiting on editor consensus label Mar 5, 2025
@g11tech g11tech closed this Mar 12, 2025
@g11tech g11tech reopened this Mar 12, 2025
@g11tech g11tech closed this Mar 19, 2025
@g11tech g11tech reopened this Mar 19, 2025
@g11tech
Copy link
Contributor

g11tech commented Mar 19, 2025

@SamWilsn not sure why the review bot is failing here

@SamWilsn
Copy link
Contributor

Neither am I. Probably because it's the template, honestly.

This'll also require changes in EIP-1, and in eipw.

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-draft This EIP is a Draft t-process
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants