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

Rough Draft #1

Merged
merged 4 commits into from
Nov 4, 2019
Merged

Rough Draft #1

merged 4 commits into from
Nov 4, 2019

Conversation

shemnon
Copy link
Owner

@shemnon shemnon commented Nov 1, 2019

Rough Draft of Validated EIP Contracts

Signed-off-by: Danno Ferrin <danno.ferrin@gmail.com>
Signed-off-by: Danno Ferrin <danno.ferrin@gmail.com>
* remove 1985 references, I don't see it applying
* enumerate invalid opcodes
* embedded links to othe EIPs
* style on backticking numbers and opcodes

Signed-off-by: Danno Ferrin <danno.ferrin@gmail.com>
@shemnon shemnon merged commit 92a6187 into master Nov 4, 2019
shemnon pushed a commit that referenced this pull request May 25, 2023
* added EIP draft for private key encapsulation

* minor updates to spec: intake function shall return the Ethereum address of the private key

* added test vector #1

* minor formatting

* minor edits

* added test vector #2 and #3, added signature verification data to #1

* changed signature to against byte values

* added test vector generator

* renamed file to assigned EIP number

* fixed file header

* updated default value for salt

* fixed offending links etc.

* fixed typo

Co-authored-by: xinbenlv <zzn@zzn.im>

* updated based on review comments

* replaced json formatting with none for better rendering

* fixed grammar

Co-authored-by: xinbenlv <zzn@zzn.im>

* fixed grammar

Co-authored-by: xinbenlv <zzn@zzn.im>

* revision suggestions taken with gratitudes

Co-authored-by: xinbenlv <zzn@zzn.im>

* revision suggestions taken with gratitudes

Co-authored-by: xinbenlv <zzn@zzn.im>

* fixed grammar

Co-authored-by: xinbenlv <zzn@zzn.im>

* fixed grammar

Co-authored-by: xinbenlv <zzn@zzn.im>

* fixed grammar

Co-authored-by: xinbenlv <zzn@zzn.im>

* fixed grammar

Co-authored-by: xinbenlv <zzn@zzn.im>

* fixed grammar

Co-authored-by: xinbenlv <zzn@zzn.im>

* fixed grammar

Co-authored-by: xinbenlv <zzn@zzn.im>

* revision suggestions taken with gratitudes

Co-authored-by: xinbenlv <zzn@zzn.im>

* fixed grammar as suggested

Co-authored-by: xinbenlv <zzn@zzn.im>

* revision suggestions taken with gratitudes

Co-authored-by: xinbenlv <zzn@zzn.im>

* fixed grammar as suggested

Co-authored-by: xinbenlv <zzn@zzn.im>

* fixed grammar as suggested

Co-authored-by: xinbenlv <zzn@zzn.im>

* fixed grammar as suggested

* fixed based on grammarly.com suggestions

* Update EIPS/eip-6051.md

Co-authored-by: Pandapip1 <45835846+Pandapip1@users.noreply.github.com>

* Update EIPS/eip-6051.md

Co-authored-by: Pandapip1 <45835846+Pandapip1@users.noreply.github.com>

* Update EIPS/eip-6051.md

Co-authored-by: Pandapip1 <45835846+Pandapip1@users.noreply.github.com>

* replacing bold fonts with links as suggested

* fixed dead links

* fixed markdown linter errors

Co-authored-by: xinbenlv <zzn@zzn.im>
Co-authored-by: Pandapip1 <45835846+Pandapip1@users.noreply.github.com>
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.

1 participant