Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Removal of master/slave words #19

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

Open
bopopescu wants to merge 1 commit into tdamdouni:master
base: master
Choose a base branch
Loading
from bopopescu:bucharest

Conversation

@bopopescu
Copy link

@bopopescu bopopescu commented Jul 24, 2020

For diversity reasons, it would be nice to try to avoid 'master' and 'slave' terminology in this repository which can be associated to slavery. The master-slave terminology could be problematic for people in several countries which has the history of slavery like Romania, USA and many others. Thank you for considering the proposal. Let me know if any changes in the PR are needed, I would be happy to implement them.

Copy link

doamne fereste..., you just find/replaced every 'master' to 'main' without any regard to what you are actually modifying. This is like painting a car red on the inside and on every piece of the motor because you want your car 'red'

TFWol reacted with laugh emoji

Copy link

TFWol commented Nov 12, 2020

@bopopescu You might want to go to the actual sources of the scripts instead; this repo is more of a collection from various locations.

Change the originals and you'll eventually see your desired change.

Copy link

itsMo07 commented Feb 13, 2021

J

Copy link

@itsMo07 itsMo07 left a comment

Choose a reason for hiding this comment

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

Thank

Copy link

@Aareon Aareon left a comment

Choose a reason for hiding this comment

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

This is the worst.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

3 more reviewers

@Aareon Aareon Aareon requested changes

@koenbnl koenbnl koenbnl approved these changes

@itsMo07 itsMo07 itsMo07 approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

AltStyle によって変換されたページ (->オリジナル) /