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

chore(docs): fix typos #1536

Merged
merged 1 commit into from Jan 23, 2024
Merged

chore(docs): fix typos #1536

merged 1 commit into from Jan 23, 2024

Conversation

Rotzbua
Copy link
Contributor

@Rotzbua Rotzbua commented Jan 23, 2024

Just in documentation.

Copy link

codecov bot commented Jan 23, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (5c2588a) 97.22% compared to head (2705862) 97.22%.

Additional details and impacted files
@@            Coverage Diff            @@
##             master    #1536   +/-   ##
=========================================
  Coverage     97.22%   97.22%           
  Complexity     2834     2834           
=========================================
  Files           175      175           
  Lines          9018     9018           
=========================================
  Hits           8768     8768           
  Misses          250      250           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Copy link
Contributor

@phil-davis phil-davis left a comment

Choose a reason for hiding this comment

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

Thanks, I obviously haven't run the spell-checker over this repo!

@@ -101,7 +101,7 @@ public function delete()
/**
* Returns the owner principal.
*
* This must be a url to a principal, or null if there's no owner
* This must be an url to a principal, or null if there's no owner
Copy link
Contributor

Choose a reason for hiding this comment

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

I guess there is "discussion" about "an url" or "a URL", when pronouncing each letter then it is "a"
"a U, an R and an L". But I will leave this change here - it does not matter either way.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Good explanation. I did not realized that both variants exist and I should leave it as it is.

@phil-davis phil-davis merged commit 960d99f into sabre-io:master Jan 23, 2024
9 checks passed
@Rotzbua Rotzbua deleted the chore_typo branch January 25, 2024 16:14
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.

None yet

2 participants