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

docs: typo fixes Update README.md #1896

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Conversation

mdqst
Copy link

@mdqst mdqst commented Dec 17, 2024

Description

Here’s a typo and a couple of other issues I noticed:

  1. "continuous" instead of "contiguous":

    • Typo: "Cairo's memory is read-only... must be continuous."
    • Correction: Replace "continuous" with "contiguous" in this context.
    • Corrected: "Cairo's memory is read-only... must be contiguous."
  2. "a yet unused":

    • Issue: Slightly awkward phrasing in "Allocation Pointer (ap) Points to a yet unused memory cell."
    • Suggested correction: Replace with "an unused memory cell."
    • Corrected: "Allocation Pointer (ap) Points to an unused memory cell."
  3. "Doesnt":

    • Typo: "Doesnt check anything for the last two."
    • Correction: Add an apostrophe for proper grammar.
    • Corrected: "Doesn't check anything for the last two."
  4. "valdation_rules":

    • Typo: "What are these valdation_rules?"
    • Correction: Fix spelling to "validation_rules."
    • Corrected: "What are these validation_rules?"
  5. "accesible_scopes":

    • Typo: "A CairoHint contains a code... and accesible_scopes."
    • Correction: Fix spelling to "accessible_scopes."
    • Corrected: "A CairoHint contains a code... and accessible_scopes."

These adjustments should improve clarity and correctness.

Checklist

  • Linked to Github Issue
  • Unit tests added
  • Integration tests added.
  • This change requires new documentation.
    • Documentation has been added/updated.
    • CHANGELOG has been updated.

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.

1 participant