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

Website: ERC-6982 Improving the comments in the interface #123

Merged
merged 5 commits into from
Dec 12, 2023

Conversation

sullof
Copy link
Contributor

@sullof sullof commented Nov 26, 2023

This PR improves the clarity of the comments to the interface, following a conversation in the community about it.

@Pandapip1 This is my fourth attempt to make a minor change to the ERC. The previous ones produced errors in existing documents not changed by me. I aligned with master and I hope this time it passes. If not, I may close it and leave the description as is.

@eip-review-bot
Copy link
Collaborator

eip-review-bot commented Nov 26, 2023

✅ All reviewers have approved.

@eip-review-bot eip-review-bot changed the title Improving the comments in the interface Website: Improving the comments in the interface Nov 26, 2023
@github-actions github-actions bot added the w-ci label Nov 26, 2023
@github-actions github-actions bot removed the w-ci label Nov 27, 2023
@sullof
Copy link
Contributor Author

sullof commented Nov 28, 2023

Should I be allowed to merge the PR?

@sullof sullof changed the title Website: Improving the comments in the interface Website: ERC-6982 Improving the comments in the interface Nov 28, 2023
Copy link

The commit 25e2a90 (as a parent of 54c04e7) contains errors.
Please inspect the Run Summary for details.

@github-actions github-actions bot added the w-ci label Nov 28, 2023
@eip-review-bot eip-review-bot enabled auto-merge (squash) December 12, 2023 15:09
Copy link
Collaborator

@eip-review-bot eip-review-bot left a 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...

@eip-review-bot eip-review-bot merged commit f5b2f32 into ethereum:master Dec 12, 2023
10 checks passed
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