Skip to content

Fix Pylint Linting Issues #5

Fix Pylint Linting Issues

Fix Pylint Linting Issues #5

Triggered via pull request January 19, 2024 18:23
Status Failure
Total duration 30s
Billable time 2m
Artifacts

checks.yml

on: pull_request
runner / pylint
17s
runner / pylint
runner / misspell
11s
runner / misspell
Fit to window
Zoom out
Zoom in

Annotations

10 errors and 10 warnings
runner / misspell: cli/ecqm-dedupe.py#L24
[misspell] reported by reviewdog 🐶 "maximisation" is a misspelling of "maximization" Raw Output: ./cli/ecqm-dedupe.py:24:49: "maximisation" is a misspelling of "maximization"
runner / misspell: cli/ecqm-dedupe.py#L28
[misspell] reported by reviewdog 🐶 "maximisation" is a misspelling of "maximization" Raw Output: ./cli/ecqm-dedupe.py:28:49: "maximisation" is a misspelling of "maximization"
runner / misspell: cli/deduplifhirLib/settings.py#L106
[misspell] reported by reviewdog 🐶 "maximisation" is a misspelling of "maximization" Raw Output: ./cli/deduplifhirLib/settings.py:106:49: "maximisation" is a misspelling of "maximization"
runner / misspell: cli/deduplifhirLib/settings.py#L110
[misspell] reported by reviewdog 🐶 "maximisation" is a misspelling of "maximization" Raw Output: ./cli/deduplifhirLib/settings.py:110:49: "maximisation" is a misspelling of "maximization"
runner / misspell: cli/deduplifhirLib/utils.py#L169
[misspell] reported by reviewdog 🐶 "maximisation" is a misspelling of "maximization" Raw Output: ./cli/deduplifhirLib/utils.py:169:49: "maximisation" is a misspelling of "maximization"
runner / misspell: cli/deduplifhirLib/utils.py#L173
[misspell] reported by reviewdog 🐶 "maximisation" is a misspelling of "maximization" Raw Output: ./cli/deduplifhirLib/utils.py:173:49: "maximisation" is a misspelling of "maximization"
runner / misspell: cli/deduplifhirLib/tests.py#L41
[misspell] reported by reviewdog 🐶 "maximisation" is a misspelling of "maximization" Raw Output: ./cli/deduplifhirLib/tests.py:41:49: "maximisation" is a misspelling of "maximization"
runner / misspell: cli/deduplifhirLib/tests.py#L45
[misspell] reported by reviewdog 🐶 "maximisation" is a misspelling of "maximization" Raw Output: ./cli/deduplifhirLib/tests.py:45:49: "maximisation" is a misspelling of "maximization"
runner / misspell: cli/deduplifhirLib/test_data.csv#L362
[misspell] reported by reviewdog 🐶 "Washignton" is a misspelling of "Washington" Raw Output: ./cli/deduplifhirLib/test_data.csv:362:41: "Washignton" is a misspelling of "Washington"
runner / misspell
reviewdog: Too many results (annotations) in diff. You may miss some annotations due to GitHub limitation for annotation created by logging command. Please check GitHub Actions log console to see all results. Limitation: - 10 warning annotations and 10 error annotations per step - 50 annotations per job (sum of annotations from all the steps) - 50 annotations per run (separate from the job annotations, these annotations aren't created by users) Source: https://github.community/t5/GitHub-Actions/Maximum-number-of-annotations-that-can-be-created-using-GitHub/m-p/39085
runner / pylint: cli/deduplifhirLib/duplicate_data_generator.py#L243
[pylint] reported by reviewdog 🐶 R1705: Unnecessary "elif" after "return", remove the leading "el" from "elif" (no-else-return) Raw Output: cli/deduplifhirLib/duplicate_data_generator.py:243:4: R1705: Unnecessary "elif" after "return", remove the leading "el" from "elif" (no-else-return)
runner / pylint: cli/deduplifhirLib/utils.py#L28
[pylint] reported by reviewdog 🐶 C0116: Missing function or method docstring (missing-function-docstring) Raw Output: cli/deduplifhirLib/utils.py:28:0: C0116: Missing function or method docstring (missing-function-docstring)
runner / pylint: cli/deduplifhirLib/utils.py#L59
[pylint] reported by reviewdog 🐶 R1732: Consider using 'with' for resource-allocating operations (consider-using-with) Raw Output: cli/deduplifhirLib/utils.py:59:11: R1732: Consider using 'with' for resource-allocating operations (consider-using-with)
runner / pylint: cli/ecqm-dedupe.py#L38
[pylint] reported by reviewdog 🐶 C0301: Line too long (133/100) (line-too-long) Raw Output: cli/ecqm-dedupe.py:38:0: C0301: Line too long (133/100) (line-too-long)
runner / pylint: cli/ecqm-dedupe.py#L1
[pylint] reported by reviewdog 🐶 C0103: Module name "ecqm-dedupe" doesn't conform to snake_case naming style (invalid-name) Raw Output: cli/ecqm-dedupe.py:1:0: C0103: Module name "ecqm-dedupe" doesn't conform to snake_case naming style (invalid-name)
runner / pylint: cli/ecqm-dedupe.py#L37
[pylint] reported by reviewdog 🐶 W0143: Comparing against a callable, did you omit the parenthesis? (comparison-with-callable) Raw Output: cli/ecqm-dedupe.py:37:7: W0143: Comparing against a callable, did you omit the parenthesis? (comparison-with-callable)
runner / pylint: cli/ecqm-dedupe.py#L1
[pylint] reported by reviewdog 🐶 R0801: Similar lines in 2 files Raw Output: cli/ecqm-dedupe.py:1:0: R0801: Similar lines in 2 files
runner / pylint: cli/ecqm-dedupe.py#L1
[pylint] reported by reviewdog 🐶 R0801: Similar lines in 2 files Raw Output: cli/ecqm-dedupe.py:1:0: R0801: Similar lines in 2 files
runner / pylint: cli/ecqm-dedupe.py#L1
[pylint] reported by reviewdog 🐶 R0801: Similar lines in 2 files Raw Output: cli/ecqm-dedupe.py:1:0: R0801: Similar lines in 2 files
runner / pylint: cli/ecqm-dedupe.py#L1
[pylint] reported by reviewdog 🐶 R0401: Cyclic import (app -> app.views -> app.utils) (cyclic-import) Raw Output: cli/ecqm-dedupe.py:1:0: R0401: Cyclic import (app -> app.views -> app.utils) (cyclic-import)