-
Notifications
You must be signed in to change notification settings - Fork 7
Description
A version of pylint is available in the Python package repositories that identifies issues with the spotfire package. Since we attempt to keep all lint issues out of the source code (either by fixing the issue identified or by disabling that message with a localized comment), this is indicative of a new check in this new version of pylint.
Please investigate these issues, and either fix the source or disable the check with a comment. Further checks by this automation will be held until this issue is closed. Make sure that the fix updates the pylint requirement in pyproject.toml (the lint key of the project.optional-dependencies section) to the version identified here (4.0.0).
For reference, here is the output of this version of pylint:
$ pylint spotfire
************* Module /home/runner/work/spotfire-python/spotfire-python/pyproject.toml
pyproject.toml:1: [E0015, ] Unrecognized option found: suggestion-mode (unrecognized-option)
************* Module spotfire.test.__main__
spotfire/test/__main__.py:31: [C0103, ] Variable name "REPORT_SUFFIX" doesn't conform to '(([a-z][a-z0-9_]{2,30})|(_[a-z0-9_]*))$' pattern ('(([a-z][a-z0-9_]{2,30})|(_[a-z0-9_]*))$' pattern) (invalid-name)
spotfire/test/__main__.py:32: [C0103, ] Variable name "REPORT_NAME" doesn't conform to '(([a-z][a-z0-9_]{2,30})|(_[a-z0-9_]*))$' pattern ('(([a-z][a-z0-9_]{2,30})|(_[a-z0-9_]*))$' pattern) (invalid-name)
spotfire/test/__main__.py:34: [C0103, ] Variable name "REPORT_SUFFIX" doesn't conform to '(([a-z][a-z0-9_]{2,30})|(_[a-z0-9_]*))$' pattern ('(([a-z][a-z0-9_]{2,30})|(_[a-z0-9_]*))$' pattern) (invalid-name)
-----------------------------------
Your code has been rated at 9.98/10
This issue was automatically opened by the pylint.yaml workflow.