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

fix: Corrected Typos Across Documentation and Code Comments #4179

Open
wants to merge 8 commits into
base: main
Choose a base branch
from

Conversation

cypherpepe
Copy link

This pull request addresses various typographical errors across multiple files in the repository. Below is a summary of the changes:

  • README.md: Corrected the hyperlink for Wormhole protocol and minor text adjustments.
  • docs/guardian.md: Fixed the term "gossipped" to "gossiped."
  • terra/docs/token_upgrades.md: Removed duplicate "the" in the sentence.
  • whitepapers/0003_token_bridge.md: Corrected duplicate "the the" and "be be."
  • whitepapers/0006_nft_bridge.md: Corrected "on per NFT" to "one per NFT."
  • whitepapers/0013_ccq.md: Fixed "guardiand watcher" to "guardian watcher."
  • wormchain/x/ibc-hooks/README.md: Corrected "Weather" to "Whether."

All changes are thoroughly verified to ensure accuracy without altering the intended meaning or functionality of the content.

Copy link
Collaborator

@SEJeff SEJeff left a comment

Choose a reason for hiding this comment

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

@cypherpepe do not tag random contributors to review PRs. This is handled by CODEOWNERS.

Thanks for the contribution. This looks good to me, but needs another reviewer.

@@ -114,7 +114,7 @@ devnet_query_request_0000000000000|

#### Solana Support

An experimental implementation of queries for Solana is being added as of January, 2024. This implementation is considered experimental because Solana does not natively support reading account data for a specific slot number, meaning each guardiand watcher will return data for its version of the most recent slot, possibly making it difficult to reach consensus. The plan is to deploy this to mainnet so that we can experiment with various ways to achieve consensus.
An experimental implementation of queries for Solana is being added as of January, 2024. This implementation is considered experimental because Solana does not natively support reading account data for a specific slot number, meaning each guardian watcher will return data for its version of the most recent slot, possibly making it difficult to reach consensus. The plan is to deploy this to mainnet so that we can experiment with various ways to achieve consensus.
Copy link
Collaborator

Choose a reason for hiding this comment

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

guardiand is the name of the executable for the guardian node software, so the original version was also correct.

@@ -1,6 +1,6 @@
<img alt="Banner" src="docs/images/banner.jpg"/>

This monorepo contains the reference implementation of the [Wormhole protocol](https://wormholenetwork.com).
This monorepo contains the reference implementation of the [Wormhole protocol](https://wormhole.com).
Copy link
Collaborator

Choose a reason for hiding this comment

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

Note for other reviewers, I very carefully ensured this url is the valid one and is not some utf magic that looks like wormhole.com and goes to an invalid website.

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.

2 participants