You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
minor #14215 Update some terminology based on AlexJS reports (wouterj)
This PR was merged into the 4.4 branch.
Discussion
----------
Update some terminology based on AlexJS reports
This PR replaces some insensitive/inconsiderate terminology based on [AlexJS](https://alexjs.com/) reports. It's very much subjective, I've not fixed all things AlexJS provided and even AlexJS (while based on official sources) implements some sort of subjective list of words to be considered insensitive/inconsiderate.
99% of the changes in this PR are:
* Removing "just", "basically", "simple", "easy" as much as possible (most of the time, removing the word was enough);
* Replacing "execute" with "run" (commands) or "call" (methods/controllers/etc)
Because sometimes AlexJS reports words that are fine in that context (and AlexJS has some trouble understanding rst syntax), I don't think we can run AlexJS as part of our GitHub actions. I did however commit the `.alexjsrc` file to ease local use. For instance, I'm using Vim with [ALE](https://github.com/dense-analysis/ale). This asynchronously runs everything I type in `*.rst` or `*.md` files through Alex; commiting the Alex config makes it easier for me to spot mistakes.
cc @lsmith77@javiereguiluz
Commits
-------
36f44c9 Update some terminology based on AlexJS reports
0 commit comments