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: fix grammar - it's -> its #565

Merged
merged 3 commits into from
Jul 1, 2024
Merged

docs: fix grammar - it's -> its #565

merged 3 commits into from
Jul 1, 2024

Conversation

jaller94
Copy link
Contributor

Title

Fix grammar: it's -> its

Description

A few places use "it's" (= it is), when they should use the possessive "its" (the dog and its ball).

Notes & open questions

Change checklist

  • I have performed a self-review of my own code
  • I have made corresponding changes to the documentation if necessary (this includes comments as well)
  • I have added tests that prove my fix is effective or that my feature works

@jaller94 jaller94 requested a review from a team as a code owner June 19, 2024 17:49
@achingbrain achingbrain changed the title Fix grammar: it's -> its docs: fix grammar - it's -> its Jul 1, 2024
Copy link
Member

@SgtPooki SgtPooki left a comment

Choose a reason for hiding this comment

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

Thanks so much for the contribution! FYI the documentation in these readme.md files are generated from the package documentation jsdoc in the index.ts files for each of the packages in the monorepo

@achingbrain achingbrain merged commit 155e24d into ipfs:main Jul 1, 2024
16 checks passed
@achingbrain
Copy link
Member

The part of the README in question is generated by aegir so I think we're all good here - ref: ipfs/aegir#1566

@achingbrain achingbrain mentioned this pull request Jul 1, 2024
@jaller94 jaller94 deleted the its branch August 10, 2024 20:15
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.

3 participants