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

EIP 665 updates #983

Merged
merged 9 commits into from
Apr 20, 2018
Merged

EIP 665 updates #983

merged 9 commits into from
Apr 20, 2018

Conversation

oberstet
Copy link
Contributor

@oberstet oberstet commented Apr 7, 2018

this brings the text in line with the 3 PRs for clients, and improves the text rgd motivation

@oberstet oberstet changed the title EIP 665 updates [WIP] EIP 665 updates Apr 7, 2018
@nicksavers
Copy link
Contributor

@oberstet As a draft this EIP seems ready to merge. If you want to tweak some things you can also update it with a PR afterwards.

@eip-automerger
Copy link

Hi! I'm a bot, and I wanted to automerge your PR, but couldn't because of the following issue(s):

@oberstet oberstet changed the title [WIP] EIP 665 updates EIP 665 updates Apr 20, 2018
@oberstet
Copy link
Contributor Author

@nicksavers yep, I added some last things, so ready for merge from my side. thanks!

@Arachnid
Copy link
Contributor

In future, if you either change the author line to contain your github username in (parentheses), or make sure your email address is public on your GitHub profile, the bot will automatically merge your pr for you.

@Arachnid Arachnid merged commit 516ce6d into ethereum:master Apr 20, 2018
@oberstet
Copy link
Contributor Author

@Arachnid aaah. thx for explaining;) I didn't get the blurbs from the bot ..

Arachnid pushed a commit to Arachnid/EIPs that referenced this pull request May 2, 2018
* correct precompile address; expand on motivation; add links to implementation PRs

* return length is 4 octets, not 1; more text polish

* typo

* expand implementation section; add link to cpp-ethereum PR
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.

4 participants