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

removed-gendered-terms #10828

Merged
merged 1 commit into from
Mar 4, 2021
Merged

removed-gendered-terms #10828

merged 1 commit into from
Mar 4, 2021

Conversation

gonyere
Copy link
Contributor

@gonyere gonyere commented Mar 4, 2021

Replacing gendered terms 'he', 'his', 'guy' and 'he/she' with 'they', 'their' and 'person'.


If this PR requires any particular action or consideration before deployment,
please check the reasons or add your own to the list:

  • Requires a database migration that can cause downtime. Postpone deployment until maintenance window[1].
  • Contains a data migration that can cause temporary inconsistency, so should be run at a specific point of time.
  • Changes some configuration files (e.g. options.yml), so the changes have to be applied manually in the reference server.
  • A new Feature Toggle[2] should be enabled in the reference server.
  • Proper documentation or announcement has to be published upfront since the introduced changes can confuse the users.

[1] https://github.com/openSUSE/open-build-service/wiki/Deployment-of-build.opensuse.org#when-there-are-migrations
[2] https://github.com/openSUSE/open-build-service/wiki/Feature-Toggles-%28Flipper%29#you-want-real-people-to-test-your-feature

@codecov
Copy link

codecov bot commented Mar 4, 2021

Codecov Report

Merging #10828 (ad4c6bf) into master (9dec0af) will decrease coverage by 0.00%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master   #10828      +/-   ##
==========================================
- Coverage   91.12%   91.12%   -0.01%     
==========================================
  Files         585      585              
  Lines       21514    21514              
==========================================
- Hits        19605    19604       -1     
- Misses       1909     1910       +1     

Copy link
Contributor

@dmarcoux dmarcoux left a comment

Choose a reason for hiding this comment

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

Thank you for the PR

@dmarcoux dmarcoux merged commit f25ffad into openSUSE:master Mar 4, 2021
@marcus-h
Copy link
Member

marcus-h commented Mar 4, 2021 via email

@dmarcoux
Copy link
Contributor

dmarcoux commented Mar 4, 2021

@marcus-h It's not a stupid question 🙂

This Wikipedia article explains well the usage of they in singular: https://en.wikipedia.org/wiki/Singular_they

@marcus-h
Copy link
Member

marcus-h commented Mar 4, 2021 via email

@dmarcoux
Copy link
Contributor

dmarcoux commented Mar 4, 2021

There is no need to call yourself stupid, I also found it strange the first time! 🙂 English is also not my native language. I'm glad to know that the article helped.

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