Skip to content
run-linting-checks

Reorganized Dependencies by Removing Unused Packages and Placing in Correct Sections #42

Sign in to view logs

Reorganized Dependencies by Removing Unused Packages and Placing in Correct Sections

Reorganized Dependencies by Removing Unused Packages and Placing in Correct Sections #42

Triggered via pull request December 14, 2023 20:09
@natalialuzuriaganatalialuzuriaga
opened #72
nat/deps_clean_up
Status Success
Total duration 48s
Artifacts –

checks.yml

on: pull_request
run-pylint
36s
run-pylint
Fit to window
Zoom out
Zoom in

Annotations

16 warnings and 34 notices
fixme: scripts/refresh_metrics.py#L34
TODO: Create a read repos-to-include.txt
bad-indentation: scripts/metricsLib/constants.py#L31
Bad indentation. Found 2 spaces, expected 4
too-many-arguments: scripts/metricsLib/metrics_data_structures.py#L51
Too many arguments (7/5)
too-many-arguments: scripts/metricsLib/metrics_data_structures.py#L145
Too many arguments (7/5)
no-else-raise: scripts/metricsLib/metrics_data_structures.py#L181
Unnecessary "else" after "raise", remove the "else" and de-indent the code inside it
too-many-arguments: scripts/metricsLib/metrics_data_structures.py#L209
Too many arguments (7/5)
too-many-arguments: scripts/metricsLib/metrics_data_structures.py#L230
Too many arguments (7/5)
too-many-arguments: scripts/metricsLib/metrics_data_structures.py#L284
Too many arguments (7/5)
too-many-arguments: scripts/metricsLib/metrics_data_structures.py#L329
Too many arguments (7/5)
fixme: scripts/metricsLib/oss_metric_entities.py#L68
TODO: should this logic be moved to the hit_metric method?
broad-exception-caught: scripts/metricsLib/oss_metric_entities.py#L164
Catching too general exception Exception
broad-exception-caught: scripts/metricsLib/oss_metric_entities.py#L317
Catching too general exception Exception
f-string-without-interpolation: scripts/metricsLib/oss_metric_entities.py#L318
Using an f-string that does not have any interpolated variables
fixme: scripts/gen_graphs.py#L65
TODO: file a pygals issue to make sparklines their own object
fixme: scripts/gen_graphs.py#L125
TODO: Just get these metrics from augur instead of storing them in json.
dangerous-default-value: scripts/gen_graphs.py#L26
Dangerous default value {} as argument
trailing-whitespace: scripts/gen_reports.py#L60
Trailing whitespace
missing-function-docstring: scripts/gen_reports.py#L31
Missing function or method docstring
missing-function-docstring: scripts/gen_reports.py#L63
Missing function or method docstring
missing-function-docstring: scripts/gen_reports.py#L69
Missing function or method docstring
trailing-whitespace: scripts/refresh_metrics.py#L47
Trailing whitespace
invalid-name: scripts/metricsLib/__init__.py#L1
Module name "metricsLib" doesn't conform to snake_case naming style
trailing-whitespace: scripts/metricsLib/constants.py#L29
Trailing whitespace
line-too-long: scripts/metricsLib/metrics_definitions.py#L105
Line too long (117/100)
line-too-long: scripts/metricsLib/metrics_definitions.py#L106
Line too long (130/100)
line-too-long: scripts/metricsLib/metrics_definitions.py#L109
Line too long (119/100)
line-too-long: scripts/metricsLib/metrics_definitions.py#L110
Line too long (131/100)
line-too-long: scripts/metricsLib/metrics_definitions.py#L113
Line too long (102/100)
line-too-long: scripts/metricsLib/metrics_definitions.py#L114
Line too long (105/100)
line-too-long: scripts/metricsLib/metrics_definitions.py#L117
Line too long (104/100)
line-too-long: scripts/metricsLib/metrics_definitions.py#L118
Line too long (106/100)
line-too-long: scripts/metricsLib/metrics_definitions.py#L146
Line too long (102/100)
trailing-whitespace: scripts/metricsLib/metrics_data_structures.py#L87
Trailing whitespace
trailing-whitespace: scripts/metricsLib/metrics_data_structures.py#L256
Trailing whitespace
trailing-whitespace: scripts/metricsLib/metrics_data_structures.py#L264
Trailing whitespace
trailing-whitespace: scripts/metricsLib/oss_metric_entities.py#L152
Trailing whitespace
line-too-long: scripts/metricsLib/oss_metric_entities.py#L160
Line too long (110/100)
trailing-whitespace: scripts/metricsLib/oss_metric_entities.py#L180
Trailing whitespace
trailing-whitespace: scripts/metricsLib/oss_metric_entities.py#L368
Trailing whitespace
trailing-whitespace: scripts/metricsLib/oss_metric_entities.py#L387
Trailing whitespace
trailing-newlines: scripts/metricsLib/oss_metric_entities.py#L394
Trailing newlines
wrong-import-order: scripts/metricsLib/oss_metric_entities.py#L11
standard import "import datetime" should be placed before "import requests"
missing-final-newline: scripts/run_pylint.py#L3
Final newline missing
missing-module-docstring: scripts/run_pylint.py#L1
Missing module docstring
line-too-long: scripts/gen_graphs.py#L72
Line too long (105/100)
trailing-whitespace: scripts/gen_graphs.py#L87
Trailing whitespace
trailing-whitespace: scripts/gen_graphs.py#L115
Trailing whitespace
missing-function-docstring: scripts/gen_graphs.py#L20
Missing function or method docstring
consider-using-f-string: scripts/gen_graphs.py#L85
Formatting a regular string which could be an f-string
line-too-long: scripts/tests/metrics_tests.py#L11
Line too long (176/100)