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

Editorial: Fix/ecip 1049 #466

Merged
merged 2 commits into from
Feb 21, 2022

Conversation

meowsbits
Copy link
Member

  • add missing License field in header
  • add missing Specification section

Per the bureaucratic regulations at ECIP Format and Structure.

Date: 2022-02-21 09:27:59-08:00
Signed-off-by: meows <[email protected]>
Date: 2022-02-21 09:33:16-08:00
Signed-off-by: meows <[email protected]>
Copy link

@rur0 rur0 left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Member

@bobsummerwill bobsummerwill left a comment

Choose a reason for hiding this comment

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

LGTM

@antsankov
Copy link
Contributor

LGTM

@gitr0n1n gitr0n1n self-requested a review February 21, 2022 22:43
Copy link
Contributor

@gitr0n1n gitr0n1n left a comment

Choose a reason for hiding this comment

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

For @bobsummerwill and his staff, I hope you find this helpful in bringing this ECIP into compliance with the ECIP process.

#465 (comment)

@bobsummerwill
Copy link
Member

I do not see why you have blocked this PR, @gitr0n1n.

These two changes from @meowsbits are free-standing and useful in their own right.

@IstoraMandiri IstoraMandiri dismissed gitr0n1n’s stale review February 21, 2022 23:45

Should not be blocked, these are minor fixes

@IstoraMandiri IstoraMandiri merged commit c3d3604 into ethereumclassic:master Feb 21, 2022
@gitr0n1n
Copy link
Contributor

I do not see why you have blocked this PR, @gitr0n1n.

These two changes from @meowsbits are free-standing and useful in their own right.

The specification did not appear to be up to ECIP-1000 standards- but it certainly is better than none. The intent wasn't to "block" but request changes.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants