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.

Sign up for GitHub

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

Jump to bottom

chore: update README.md #870

Merged
merged 1 commit into from
Jun 27, 2023
Merged

chore: update README.md #870

merged 1 commit into from
Jun 27, 2023

Conversation

omniwired
Copy link
Contributor

@omniwired omniwired commented Jun 6, 2023 •
edited by zivkovicmilos

Format and wording changes

Description

Please provide a detailed description of the changes made in this pull request.

Contributors Checklist

  • Added new tests, or not needed, or not feasible
  • Provided an example (e.g. screenshot) to aid review or the PR is self-explanatory
  • Updated the official documentation or not needed
  • No breaking changes were made, or a BREAKING CHANGE: xxx message was included in the description
  • Added references to related issues and PRs
  • Provided any useful hints for running manual tests

Maintainers Checklist

  • Checked that the author followed the guidelines in CONTRIBUTING.md
  • Checked the conventional-commit (especially PR title and verb, presence of BREAKING CHANGE: in the body)
  • Ensured that this PR is not a significant change or confirmed that the review/consideration process was appropriate for the change

@omniwired
Update README.md
5f7770b 
Format and wording changes
@omniwired omniwired requested a review from a team as a code owner June 6, 2023 04:17
@github-actions github-actions bot added the 📦 🌐 tendermint v2 Issues or PRs tm2 related label Jun 6, 2023
@zivkovicmilos zivkovicmilos requested a review from moul June 16, 2023 10:44
@zivkovicmilos zivkovicmilos changed the title Update README.md chore: update README.md Jun 16, 2023
zivkovicmilos
zivkovicmilos approved these changes Jun 16, 2023
View reviewed changes
Copy link
Member

@zivkovicmilos zivkovicmilos 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 for the contribution 🙏

I think the README looks cleaner now, but it may be just my personal preference

What do you think @moul ?

@zxxma zxxma mentioned this pull request Jun 22, 2023
Open
@omniwired
Copy link
Contributor Author

bump @moul anything else needed for this one?

@moul moul merged commit abf7792 into gnolang:master Jun 27, 2023
6 of 13 checks passed
Doozers pushed a commit to Doozers/gno that referenced this pull request Aug 31, 2023
@omniwired @Doozers
chore: update README.md (gnolang#870)
af6e270
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@moul moul moul approved these changes

@zivkovicmilos zivkovicmilos zivkovicmilos approved these changes

Assignees

@omniwired omniwired

Labels
📦 🌐 tendermint v2 Issues or PRs tm2 related
Projects
TM2/GnoVM
Status: Done
🚀 The Launch
Status: 🔵 Not Needed for Launch
🧐 Review Meeting
Archived in project
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@omniwired @moul @zivkovicmilos