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

Switch to more inclusive language #14734

Open
core-ai-bot opened this issue Aug 31, 2021 · 3 comments
Open

Switch to more inclusive language #14734

core-ai-bot opened this issue Aug 31, 2021 · 3 comments

Comments

@core-ai-bot
Copy link
Member

Issue by alb
Sunday Jul 19, 2020 at 01:23 GMT
Originally opened as adobe/brackets#15188


This pull requests implements the changes suggested in #15176.

Replaced:

  • blacklist with blocklist
  • whitelist with allow-list
  • slave with subordinate

@jha-g@narayani28 please review


alb included the following code: https://github.com/adobe/brackets/pull/15188/commits

@core-ai-bot
Copy link
Member Author

Comment by alb
Sunday Jul 19, 2020 at 01:33 GMT


It appears the build failed due to an issue with phantomjs not installing. This doesn't appear to be an issue with the pull request but rather how phantomjs is being installed.

npm ERR! phantomjs@1.9.18 install: `node install.js`

npm ERR! Exit status 1

npm ERR! 

npm ERR! Failed at the phantomjs@1.9.18 install script 'node install.js'.

npm ERR! Make sure you have the latest version of node.js and npm installed.

npm ERR! If you do, this is most likely a problem with the phantomjs package,

npm ERR! not with npm itself.

npm ERR! Tell the author that this fails on your system:

npm ERR!     node install.js

npm ERR! You can get information on how to open an issue for this project with:

npm ERR!     npm bugs phantomjs

npm ERR! Or if that isn't available, you can get their info via:

npm ERR!     npm owner ls phantomjs

npm ERR! There is likely additional logging output above.

npm ERR! Please include the following file with any support request:

npm ERR!     /home/travis/build/adobe/brackets/npm-debug.log

npm ERR! code 1

The command "eval npm install  " failed 3 times.

The command "npm install " failed and exited with 1 during .

@core-ai-bot
Copy link
Member Author

Comment by ferdnyc
Monday Jul 20, 2020 at 09:41 GMT


(Disclaimer: I'm not a project developer or... well, anyone, really, in the context of the Brackets codebase. So, my butting-in carries less than no weight here. Feel free to ignore me completely.)

Might be worth squashing the branch,@alb — no reason a change like this has to happen across two separate commits.

@core-ai-bot
Copy link
Member Author

Comment by alb
Monday Jul 20, 2020 at 14:19 GMT


@ferdnyc thanks for the advice 😄 I have squashed the branch, according to your advice.

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

No branches or pull requests

1 participant