Skip to content

Fixing typo occurrences#977

Merged
eriqua merged 1 commit intomainfrom
users/erikag/typo-fix
Feb 6, 2022
Merged

Fixing typo occurrences#977
eriqua merged 1 commit intomainfrom
users/erikag/typo-fix

Conversation

@eriqua
Copy link
Copy Markdown
Contributor

@eriqua eriqua commented Feb 6, 2022

Change

Small update to fix parameter description typo

Type of Change

Please delete options that are not relevant.

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • This change requires a documentation update (Wiki)

Checklist

  • I'm sure there are no other open Pull Requests for the same update/change
  • My corresponding pipelines / checks run clean and green without any errors or warnings
  • My code follows the style guidelines of this project
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation (readme)
  • I did format my code

@eriqua eriqua self-assigned this Feb 6, 2022
@eriqua eriqua enabled auto-merge (squash) February 6, 2022 14:34
@eriqua
Copy link
Copy Markdown
Contributor Author

eriqua commented Feb 6, 2022

ht be missing the point to this PR: How is it's a typo, and its being correct?

@MariusStorhaug 😄 I'm not a native but they are two different things. I believe the intention of the updated sentences was to use its as the possessive form of the it pronoun and not it's as the contraction of it is.

Copy link
Copy Markdown
Contributor

@MariusStorhaug MariusStorhaug left a comment

Choose a reason for hiding this comment

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

@eriqua eriqua merged commit 5cd25b2 into main Feb 6, 2022
@eriqua eriqua deleted the users/erikag/typo-fix branch February 6, 2022 16:26
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