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

Suggest to avoid "Master branch" instead of "Default branch" #1496

Merged
merged 2 commits into from
Nov 23, 2020

Conversation

ba32107
Copy link
Contributor

@ba32107 ba32107 commented Nov 20, 2020

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:

  • All of the tests are passing.
  • I have reviewed my changes in staging. (look for the deploy-to-heroku link in your pull request, then click View deployment)
  • For content changes, I have reviewed the localization checklist
  • For content changes, I have reviewed the Content style guide for GitHub Docs.

@ba32107 ba32107 marked this pull request as ready for review November 20, 2020 10:49
@janiceilene
Copy link
Contributor

@ba32107 Thanks so much for opening a PR! I'll get this triaged for review ✨

@janiceilene janiceilene added the contributing docs Content related to our contributing docs label Nov 20, 2020
Copy link
Member

@emilyistoofunky emilyistoofunky left a 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 |
Copy link
Member

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!

@emilyistoofunky emilyistoofunky merged commit e0cae0a into github:main Nov 23, 2020
@github-actions
Copy link
Contributor

Thanks very much for contributing! Your pull request has been merged 🎉 You should see your changes appear on the site in approximately 24 hours.

@ba32107 ba32107 deleted the patch-2 branch November 24, 2020 07:36
jnidzwetzki pushed a commit to jnidzwetzki/docs that referenced this pull request Oct 6, 2022
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
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
contributing docs Content related to our contributing docs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Content style guide suggests to avoid the wrong term
4 participants