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 grammar clarity #1283

Merged
merged 3 commits into from
Sep 9, 2024

Conversation

rolandolopez36
Copy link
Contributor

Summary of Changes

  1. Improved clarity:

    • Simplified sentence structures to make the content more concise and easier to read.
    • Replaced phrases like "If you face any issues" with "If you encounter any issues" for more formal and professional language.
  2. Grammar corrections:

    • Corrected small grammatical issues, such as replacing “then simply follow” with “simply follow,” ensuring consistent tone throughout.
  3. Enhanced flow and consistency:

    • Reorganized some sentences to improve the overall flow of the document, especially in the "Using Foundryup" and "Building from Source" sections.
    • Standardized the formatting for technical instructions and commands to maintain consistency across the document.
  4. Better emphasis and formatting:

    • Improved the formatting of the "Note" sections, ensuring proper spacing and clear emphasis on important information (e.g., using Git BASH or WSL on Windows).
    • Made use of bullet points where needed for easier reading, especially in step-by-step processes.
  5. Clarified Docker section:

    • Reworded the Docker section for clarity, particularly in describing how to download and build images. Added small clarifications to avoid confusion for first-time users.

@mattsse mattsse merged commit 02833bc into foundry-rs:master Sep 9, 2024
1 check passed
@rolandolopez36 rolandolopez36 deleted the improve-grammar-clarity branch September 9, 2024 13:17
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