-
Notifications
You must be signed in to change notification settings - Fork 92
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
Use inclusive words and update pylint.config #548
Comments
The non-inclusive words in pylint.config are a result of the imported pylint code. The change would first need to be made in that repo: https://github.com/GoogleCloudPlatform/python-repo-tools |
Thanks for pointing where it comes from! Never seen it for myself until now 😅 Origin: https://github.com/GoogleCloudPlatform/python-repo-tools/blob/master/gcp_devrel/tools/pylint.py |
Thank you for opening a Pull Request! Before submitting your PR, there are a few things you can do to make sure it goes smoothly: - [x] Make sure to open an issue as a [bug/issue](https://github.com/googleapis/python-spanner/issues/new/choose) before writing your code! That way we can discuss the change, evaluate designs, and agree on the general idea - [ ] Ensure the tests and linter pass - [ ] Code coverage does not decrease (if any source code was changed) - [ ] Appropriate docs were updated (if necessary) Unsure how to update https://github.com/googleapis/python-spanner/blob/master/pylint.config.py#L25-L28, filed #548 to keep track of it. Fixes #547 🦕
@larkee, @dandhlee We no longer actually use |
* chore: remove fossils of pylint Closes #548. * 🦉 Updates from OwlBot See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com>
There are mentions of non-inclusive words in pylint.config. Not sure how to update this file, please update it from
master
tomain
or other applicable inclusive terms. Please also update any other mentions of non-inclusive terms in the library proto or in this repository in general!The text was updated successfully, but these errors were encountered: