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 ERC-1363: Move motivation and rationale to eip1 recommendation #74

Conversation

vittominacori
Copy link
Contributor

@vittominacori vittominacori commented Oct 31, 2023

This PR is part of some changes I would like to make on ERC1363 to follow the modern EIP1 recommendation.

This PR:

  • Update preamble and removed the not necessary simple summary section
  • Refactor abstract section
  • Moves the rationale from motivation into the rationale section

Note that this PR will not pass the CI, because of many changes required by validator and that are not addressed here but will be addressed in other PR.

I.g. external links, broken link, new line after h1, etc.

Other PR will address other sections of ERC1363

@eip-review-bot
Copy link
Collaborator

eip-review-bot commented Oct 31, 2023

File ERCS/erc-1363.md

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

@eip-review-bot eip-review-bot changed the title Move motivation and rationale to eip1 recommendation Update ERC-1363: Move motivation and rationale to eip1 recommendation Oct 31, 2023
@github-actions github-actions bot added the w-ci label Oct 31, 2023
@eip-review-bot eip-review-bot changed the title Update ERC-1363: Move motivation and rationale to eip1 recommendation Update ERC-1363: Move motivation and rationale to eip1 recommendation Nov 18, 2023
@github-actions github-actions bot added w-ci and removed w-ci labels Nov 18, 2023
Copy link

There has been no activity on this pull request for 2 weeks. It will be closed after 3 months of inactivity. If you would like to move this PR forward, please respond to any outstanding feedback or add a comment indicating that you have addressed all required feedback and are ready for a review.

@github-actions github-actions bot removed the w-stale label Dec 23, 2023
Copy link

The commit 20ee1ff (as a parent of 41a72a1) contains errors.
Please inspect the Run Summary for details.

@SamWilsn
Copy link
Collaborator

SamWilsn commented Jan 3, 2024

These changes are a bit more involved than we allow for final EIPs. I think the best option would be to open a new EIP with the updated content.

@SamWilsn SamWilsn closed this Jan 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants