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

Update pylint configuration #1640

Open
wants to merge 3 commits into
base: develop
Choose a base branch
from

Conversation

pjonsson
Copy link
Contributor

@pjonsson pjonsson commented Sep 21, 2024

Reason for this pull request

There are many warnings in the pylint CI job, where the majority are about removed checks. Fix most of them by removing the checks from pylintrc.

Proposed changes

  • Closes #xxxx

  • Tests added / passed

  • Fully documented, including docs/about/whats_new.rst for all changes


📚 Documentation preview 📚: https://datacube-core--1640.org.readthedocs.build/en/1640/

These options all give a warning on the form:

R0022: Useless option value for '--enable', 'deprecated-urllib-function' was removed from pylint, see pylint-dev/pylint#4942. (useless-option-value)
These options give a warning:

W0012: Unknown option value for '--enable', expected a valid pylint message and got 'long-suffix' (unknown-option-value)

so they have presumably been removed
from pylint since the last pylintrc
update in 2019.
The rule has been removed from pylint,
so remove the disable line.
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.

1 participant