-
Notifications
You must be signed in to change notification settings - Fork 5.3k
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-4910: Move to Final #6843
Update EIP-4910: Move to Final #6843
Conversation
✅ All reviewers have approved. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please fix the markdown lint errors.
The Jekyll bot fails due to a missing HTML file. That is on the github side though, I think. I have a question: If I cannot use inline HTML how can I make a requirement externally referencable through a URL? If I use e.g. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@lightclient I'm okay with the inline HTML in this proposal, but I don't want to clear your review in case you strongly object.
There was a problem hiding this 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...
There was a problem hiding this 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...
There was a problem hiding this 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...
There was a problem hiding this 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...
There was a problem hiding this 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...
There was a problem hiding this 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...
There was a problem hiding this 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...
There was a problem hiding this 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...
There was a problem hiding this 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...
There was a problem hiding this 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...
@lightclient I think you'll need to dismiss your review? |
Head branch was pushed to by a user without write access
ffd220e
There was a problem hiding this 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...
There was a problem hiding this 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...
The commit 14f08a1 (as a parent of 1c44d6c) contains errors. |
* Initial submission of L2 Token List Draft * updated headers (cut and paste error) * fixing cli errors * fixing cli errors * fix cli errors * fixing cli errors * fixing cli errors * fix cli errors * fix cli errors * fixing cli errors * fix cli errors * fixing cli errors * fixing final cli error * updated file name etc. to PR number * Fixing markdown linter errors @Pandapip1 -- the issue is now the same with inline HTM as with #6843. Please, approve if this is ok. I had asked @lightclient for an alternative to make individual requirements referenceable through a URL, apart from inline or using ##### which is not allowed either. * fixing some more markdown linter errors * fixing markdown linter errors * fixing one markdown error apart from inline * Commit from EIP-Bot * fixing unexplained file naming and EIP number errors * Update EIPS/eip-6734.md Co-authored-by: Sam Wilson <[email protected]> * Update EIPS/eip-6734.md Co-authored-by: Sam Wilson <[email protected]> * Update EIPS/eip-6734.md Co-authored-by: Sam Wilson <[email protected]> * Updates per request from @SamWilsn * correcting CI issues * fixed error in rfc links * fixing CI issues * Update EIPS/eip-6734.md Co-authored-by: Sam Wilson <[email protected]> * Update EIPS/eip-6734.md Co-authored-by: Sam Wilson <[email protected]> * Update EIPS/eip-6734.md Co-authored-by: Sam Wilson <[email protected]> * updating PRs based on @SamWilsn requests * Update EIPS/eip-6734.md --------- Co-authored-by: eth-bot <[email protected]> Co-authored-by: Sam Wilson <[email protected]>
* Initial submission of L2 Token List Draft * updated headers (cut and paste error) * fixing cli errors * fixing cli errors * fix cli errors * fixing cli errors * fixing cli errors * fix cli errors * fix cli errors * fixing cli errors * fix cli errors * fixing cli errors * fixing final cli error * updated file name etc. to PR number * Fixing markdown linter errors @Pandapip1 -- the issue is now the same with inline HTM as with ethereum#6843. Please, approve if this is ok. I had asked @lightclient for an alternative to make individual requirements referenceable through a URL, apart from inline or using ##### which is not allowed either. * fixing some more markdown linter errors * fixing markdown linter errors * fixing one markdown error apart from inline * Commit from EIP-Bot * fixing unexplained file naming and EIP number errors * Update EIPS/eip-6734.md Co-authored-by: Sam Wilson <[email protected]> * Update EIPS/eip-6734.md Co-authored-by: Sam Wilson <[email protected]> * Update EIPS/eip-6734.md Co-authored-by: Sam Wilson <[email protected]> * Updates per request from @SamWilsn * correcting CI issues * fixed error in rfc links * fixing CI issues * Update EIPS/eip-6734.md Co-authored-by: Sam Wilson <[email protected]> * Update EIPS/eip-6734.md Co-authored-by: Sam Wilson <[email protected]> * Update EIPS/eip-6734.md Co-authored-by: Sam Wilson <[email protected]> * updating PRs based on @SamWilsn requests * Update EIPS/eip-6734.md --------- Co-authored-by: eth-bot <[email protected]> Co-authored-by: Sam Wilson <[email protected]>
* Initial submission of L2 Token List Draft * updated headers (cut and paste error) * fixing cli errors * fixing cli errors * fix cli errors * fixing cli errors * fixing cli errors * fix cli errors * fix cli errors * fixing cli errors * fix cli errors * fixing cli errors * fixing final cli error * updated file name etc. to PR number * Fixing markdown linter errors @Pandapip1 -- the issue is now the same with inline HTM as with ethereum#6843. Please, approve if this is ok. I had asked @lightclient for an alternative to make individual requirements referenceable through a URL, apart from inline or using ##### which is not allowed either. * fixing some more markdown linter errors * fixing markdown linter errors * fixing one markdown error apart from inline * Commit from EIP-Bot * fixing unexplained file naming and EIP number errors * Update EIPS/eip-6734.md Co-authored-by: Sam Wilson <[email protected]> * Update EIPS/eip-6734.md Co-authored-by: Sam Wilson <[email protected]> * Update EIPS/eip-6734.md Co-authored-by: Sam Wilson <[email protected]> * Updates per request from @SamWilsn * correcting CI issues * fixed error in rfc links * fixing CI issues * Update EIPS/eip-6734.md Co-authored-by: Sam Wilson <[email protected]> * Update EIPS/eip-6734.md Co-authored-by: Sam Wilson <[email protected]> * Update EIPS/eip-6734.md Co-authored-by: Sam Wilson <[email protected]> * updating PRs based on @SamWilsn requests * Update EIPS/eip-6734.md --------- Co-authored-by: eth-bot <[email protected]> Co-authored-by: Sam Wilson <[email protected]>
* Initial submission of L2 Token List Draft * updated headers (cut and paste error) * fixing cli errors * fixing cli errors * fix cli errors * fixing cli errors * fixing cli errors * fix cli errors * fix cli errors * fixing cli errors * fix cli errors * fixing cli errors * fixing final cli error * updated file name etc. to PR number * Fixing markdown linter errors @Pandapip1 -- the issue is now the same with inline HTM as with ethereum#6843. Please, approve if this is ok. I had asked @lightclient for an alternative to make individual requirements referenceable through a URL, apart from inline or using ##### which is not allowed either. * fixing some more markdown linter errors * fixing markdown linter errors * fixing one markdown error apart from inline * Commit from EIP-Bot * fixing unexplained file naming and EIP number errors * Update EIPS/eip-6734.md Co-authored-by: Sam Wilson <[email protected]> * Update EIPS/eip-6734.md Co-authored-by: Sam Wilson <[email protected]> * Update EIPS/eip-6734.md Co-authored-by: Sam Wilson <[email protected]> * Updates per request from @SamWilsn * correcting CI issues * fixed error in rfc links * fixing CI issues * Update EIPS/eip-6734.md Co-authored-by: Sam Wilson <[email protected]> * Update EIPS/eip-6734.md Co-authored-by: Sam Wilson <[email protected]> * Update EIPS/eip-6734.md Co-authored-by: Sam Wilson <[email protected]> * updating PRs based on @SamWilsn requests * Update EIPS/eip-6734.md --------- Co-authored-by: eth-bot <[email protected]> Co-authored-by: Sam Wilson <[email protected]>
* Initial submission of L2 Token List Draft * updated headers (cut and paste error) * fixing cli errors * fixing cli errors * fix cli errors * fixing cli errors * fixing cli errors * fix cli errors * fix cli errors * fixing cli errors * fix cli errors * fixing cli errors * fixing final cli error * updated file name etc. to PR number * Fixing markdown linter errors @Pandapip1 -- the issue is now the same with inline HTM as with ethereum#6843. Please, approve if this is ok. I had asked @lightclient for an alternative to make individual requirements referenceable through a URL, apart from inline or using ##### which is not allowed either. * fixing some more markdown linter errors * fixing markdown linter errors * fixing one markdown error apart from inline * Commit from EIP-Bot * fixing unexplained file naming and EIP number errors * Update EIPS/eip-6734.md Co-authored-by: Sam Wilson <[email protected]> * Update EIPS/eip-6734.md Co-authored-by: Sam Wilson <[email protected]> * Update EIPS/eip-6734.md Co-authored-by: Sam Wilson <[email protected]> * Updates per request from @SamWilsn * correcting CI issues * fixed error in rfc links * fixing CI issues * Update EIPS/eip-6734.md Co-authored-by: Sam Wilson <[email protected]> * Update EIPS/eip-6734.md Co-authored-by: Sam Wilson <[email protected]> * Update EIPS/eip-6734.md Co-authored-by: Sam Wilson <[email protected]> * updating PRs based on @SamWilsn requests * Update EIPS/eip-6734.md --------- Co-authored-by: eth-bot <[email protected]> Co-authored-by: Sam Wilson <[email protected]>
* Initial submission of L2 Token List Draft * updated headers (cut and paste error) * fixing cli errors * fixing cli errors * fix cli errors * fixing cli errors * fixing cli errors * fix cli errors * fix cli errors * fixing cli errors * fix cli errors * fixing cli errors * fixing final cli error * updated file name etc. to PR number * Fixing markdown linter errors @Pandapip1 -- the issue is now the same with inline HTM as with ethereum#6843. Please, approve if this is ok. I had asked @lightclient for an alternative to make individual requirements referenceable through a URL, apart from inline or using ##### which is not allowed either. * fixing some more markdown linter errors * fixing markdown linter errors * fixing one markdown error apart from inline * Commit from EIP-Bot * fixing unexplained file naming and EIP number errors * Update EIPS/eip-6734.md Co-authored-by: Sam Wilson <[email protected]> * Update EIPS/eip-6734.md Co-authored-by: Sam Wilson <[email protected]> * Update EIPS/eip-6734.md Co-authored-by: Sam Wilson <[email protected]> * Updates per request from @SamWilsn * correcting CI issues * fixed error in rfc links * fixing CI issues * Update EIPS/eip-6734.md Co-authored-by: Sam Wilson <[email protected]> * Update EIPS/eip-6734.md Co-authored-by: Sam Wilson <[email protected]> * Update EIPS/eip-6734.md Co-authored-by: Sam Wilson <[email protected]> * updating PRs based on @SamWilsn requests * Update EIPS/eip-6734.md --------- Co-authored-by: eth-bot <[email protected]> Co-authored-by: Sam Wilson <[email protected]>
When opening a pull request to submit a new EIP, please use the suggested template: https://github.com/ethereum/EIPs/blob/master/eip-template.md
We have a GitHub bot that automatically merges some PRs. It will merge yours immediately if certain criteria are met: