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

Spell out mission statement, fixes #1128 #4748

Merged
merged 6 commits into from
Feb 12, 2022

Conversation

fulldecent
Copy link
Contributor

Related to EIPIP meeting https://github.com/ethereum-cat-herders/EIPIP/issues/101#issuecomment-1011094223


Please accept this PR and also update the repository description from:

The Ethereum Improvement Proposal repository

to

This repository tracks the ongoing improvements to Ethereum


This PR takes inspiration from https://github.com/apple/swift-evolution

@eth-bot
Copy link
Collaborator

eth-bot commented Feb 2, 2022

All tests passed; auto-merging...

(pass) README.md

classification
ambiguous
  • file README.md is not a valid filename, but this error has been ignored due to editor approvals

Copy link
Member

@lightclient lightclient left a comment

Choose a reason for hiding this comment

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

Probably best to remove this whitespace at the end, otherwise the change looks good. Travis CI is failing and atm can't load to see why. Would also be good to have @axic or @MicahZoltu lay eyes on this.

README.md Outdated Show resolved Hide resolved
fulldecent and others added 2 commits February 8, 2022 11:47
Co-authored-by: lightclient <14004106+lightclient@users.noreply.github.com>
@fulldecent
Copy link
Contributor Author

Thank you. Rebased. Maybe that will jiggle it into being accepted by the robots.

```sh
$ gem install bundler
```
```sh
Copy link
Member

Choose a reason for hiding this comment

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

Nit: why do these need to be indented?

Copy link
Contributor Author

@fulldecent fulldecent Feb 8, 2022

Choose a reason for hiding this comment

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

This renders so that the code fence is part of the step.

Without indenting, the code fence is not semantically part of the list of instructions.

Copy link
Member

Choose a reason for hiding this comment

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

Ah!

Before: Screenshot

After: Screenshot

README.md Outdated

Ethereum Improvement Proposals (EIPs) describe standards for the Ethereum platform, including core protocol specifications, client APIs, and contract standards. **Browse all current and draft EIPs on [the official EIP site](https://eips.ethereum.org/).**
This repository tracks the ongoing improvements to Ethereum. It contains:
Copy link
Member

Choose a reason for hiding this comment

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

I think "tracks" is a very loose term here and could be understood differently, i.e. "this is the sole place to get information about what is being worked on for Ethereum", which is clearly the wrong impression to give.

Copy link
Contributor Author

@fulldecent fulldecent Feb 8, 2022

Choose a reason for hiding this comment

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

Remove definite article at 970c2a9.

Now we're tracking improvements instead of tracking (all of) the improvements.

Copy link
Member

Choose a reason for hiding this comment

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

I can't say it is perfect, but I have no good suggestions.

README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
fulldecent and others added 3 commits February 8, 2022 12:05
Co-authored-by: Alex Beregszaszi <alex@rtfs.hu>
Co-authored-by: Alex Beregszaszi <alex@rtfs.hu>
Copy link
Member

@axic axic left a comment

Choose a reason for hiding this comment

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

It is an improvement from my perspective.

README.md Outdated Show resolved Hide resolved
Co-authored-by: lightclient <14004106+lightclient@users.noreply.github.com>
@axic axic dismissed lightclient’s stale review February 11, 2022 08:50

Change was applied.

@MicahZoltu
Copy link
Contributor

Looks like we have tentative approval from multiple editors, so I'm going to merge this.

@MicahZoltu MicahZoltu merged commit 301346d into ethereum:master Feb 12, 2022
PowerStream3604 pushed a commit to PowerStream3604/EIPs that referenced this pull request May 19, 2022
* Spell out mission statement, fixes ethereum#1128

* Update README.md

Co-authored-by: lightclient <14004106+lightclient@users.noreply.github.com>

* Remove definite article against "tracks"

* Update README.md

Co-authored-by: Alex Beregszaszi <alex@rtfs.hu>

* Update README.md

Co-authored-by: Alex Beregszaszi <alex@rtfs.hu>

* Update README.md

Co-authored-by: lightclient <14004106+lightclient@users.noreply.github.com>

Co-authored-by: lightclient <14004106+lightclient@users.noreply.github.com>
Co-authored-by: Alex Beregszaszi <alex@rtfs.hu>
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.

5 participants