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 typos #123

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

Fix typos #123

wants to merge 6 commits into from

Conversation

Hopium21
Copy link

@Hopium21 Hopium21 commented Mar 2, 2025

Grammar mistakes have been corrected and readability improved.

Copy link
Contributor

juntao commented Mar 2, 2025

Hello, I am a PR summary agent on flows.network. Here are my reviews of code commits in this PR.


Overall Summary

Potential Issues and Errors

  1. Grammar Correction in docs/litepaper.md: The change from "the API service are" to "the API service for" at line 146 seems incorrect as it doesn't form a complete grammatically correct sentence. It should likely be changed to either "the API services are" or "the API service is", depending on the context.

Most Important Findings

  1. Consistent Formatting: Multiple patches address file formatting issues, such as ensuring proper spacing at the end of lines and removing no newline at end of file markers. This improves overall code readability.
  2. Grammar Fixes: Corrections have been made in various files to improve grammatical accuracy, such as changing "too much resources" to "too many resources", "Checkout" to "Check out", and removing unnecessary punctuation.
  3. Typo Corrections: Several typos across different files were corrected, including fixing "mis-configured" to "misconfigured" and "Pre-requisites" to "Prerequisites". These changes enhance the clarity and professionalism of the documentation.

Details

Commit 7344e5e5cc8ed96f4e59e56692a09f048d379cbe

-Key Changes:

  1. Fixed Typo in intro.md: Corrected "Checkout" to "Check out" for better grammar and readability.

Commit d8296f38b028cf114787bb326739982bfcdb747b

-Key Changes:

  1. Removed unnecessary comma after "Asia Pacific (Osaka)" in the description of available regions.
  2. Fixed missing space at the end of the file to ensure proper formatting.

Commit a612c5b4b826e920fa25a0fa6eafc0c7ad7f6ddb

-### Key Changes:

  1. Grammar Correction: Changed "too much resources" to "too many resources".
  2. Formatting Fix: Removed the no newline at end of file marker for consistency.

Commit cdd0efcf523617c772535e848cafbb20e07578d1

Key Changes:

  1. Corrected Typo: Changed "Pre-requisites" to "Prerequisites".
  2. Removed Trailing Whitespace: Fixed a trailing whitespace issue at the end of the file.

Commit 5004d09eb6b6cc781bfc44c85072c5cb8dcb1d1e

-1. Corrected the typo from "mis-configured" to "misconfigured" in troubleshooting.md.

Commit 9834a4bdacbc2a60f795bc2f780a563029cded1c

  • Key Changes:
  • Corrected the typo "the API service are" to "the API service for" in docs/litepaper.md at line 146.

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