-
Notifications
You must be signed in to change notification settings - Fork 59.8k
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
Suggest to avoid "Master branch" instead of "Default branch" #1496
Conversation
@ba32107 Thanks so much for opening a PR! I'll get this triaged for review ✨ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for letting us know about this @ba32107! ⚡ I'll get this merged.
@@ -162,7 +162,7 @@ It’s critical that all of our documentation is inclusive and respectful of our | |||
| --- | --- | | |||
| Allowlist | Whitelist | | |||
| Denylist | Blacklist | | |||
| Main branch | Default branch | | |||
| Default/Main branch | Master branch | |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It looks like we made a typo here - thanks for catching it!
Thanks very much for contributing! Your pull request has been merged 🎉 You should see your changes appear on the site in approximately 24 hours. |
The Timescale Cloud excerpt was defined twice, and the less-descriptive definition was being used. Remove the duplicate definition and refine the remaining one to better promote the benefits of Cloud
Why:
Fixes #1495
What's being changed:
I've changed the suggestion to avoid "Default branch" to avoid "Master branch". I believe the latter can be considered offending, whereas I don't see any issue with the term "Default branch".
Check off the following: