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

Improve readme style #3577

Merged
merged 7 commits into from Jul 25, 2022
Merged

Improve readme style #3577

merged 7 commits into from Jul 25, 2022

Conversation

krakxn
Copy link
Contributor

@krakxn krakxn commented Jul 25, 2022

Category

Documentation

Context

Fixed grammatical errors in README.md (accounts for minor changes, structuring etc.).

Cheers!

PR Checklist

  • Tests
  • Documentation
  • Changelog entry

Copy link
Contributor

@frangio frangio left a comment

Choose a reason for hiding this comment

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

These are mostly stylistic rather than grammar as far as I can tell. Some seem rather opinionated and I've suggested to roll them back.

README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
krakxn and others added 6 commits July 25, 2022 22:41
Co-authored-by: Francisco <frangio.1@gmail.com>
Co-authored-by: Francisco <frangio.1@gmail.com>
Co-authored-by: Francisco <frangio.1@gmail.com>
Co-authored-by: Francisco <frangio.1@gmail.com>
Co-authored-by: Francisco <frangio.1@gmail.com>
Copy link
Contributor

@frangio frangio left a comment

Choose a reason for hiding this comment

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

Thank you!

@frangio frangio changed the title Fix grammatical errors Improve readme style Jul 25, 2022
@frangio frangio merged commit 48dd186 into OpenZeppelin:master Jul 25, 2022
ronhuafeng added a commit to ronhuafeng/openzeppelin-contracts that referenced this pull request Sep 9, 2022
Co-authored-by: Francisco <frangio.1@gmail.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.

None yet

2 participants