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 Typographical Errors #935

Closed
wants to merge 4 commits into from
Closed

Conversation

Pricstas
Copy link

This pull request addresses several typographical errors and stylistic improvements in the documentation files howitworks.md and what-is-semaphore.md.

Changes Made:

  1. Corrected grammatical usage in descriptions:
    • Replaced "comprised of" with "comprises" for accuracy.
    • Adjusted phrasing for clarity (e.g., "at the same booth" instead of "a the same booth").
  2. Improved conciseness by replacing verbose expressions:
    • Changed "in a unique way" to "uniquely."

These updates improve the overall readability and precision of the documentation.

Related Issues

No open issues are linked to this pull request. The changes are general improvements identified during documentation review.

Checklist

  • My code follows the style guidelines of this project.
  • I have performed a self-review of my changes.
  • All modified content was tested for clarity and correctness.
  • No new warnings were generated as a result of this update.
  • I ran yarn format and yarn lint to ensure code quality.

@vplasencia
Copy link
Member

Thank you for submitting this PR! While we greatly appreciate your effort, we will be closing it to align with our contribution guidelines. As stated in the guidelines: "We do not accept minor grammatical fixes (e.g., correcting typos, rewording sentences) unless they significantly improve clarity in technical documentation."

We truly value your interest in contributing to the project and encourage you to explore our open issues. We look forward to seeing your future contributions!

@vplasencia vplasencia closed this Jan 15, 2025
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