Skip to content
run-linting-checks

Make Site Work Better with Augur and Add Working Desired Endpoints #41

Sign in to view logs

Make Site Work Better with Augur and Add Working Desired Endpoints

Make Site Work Better with Augur and Add Working Desired Endpoints #41

Triggered via pull request December 13, 2023 22:40
@IsaacMilarkyIsaacMilarky
synchronize #71
more-augur-metrics
Status Success
Total duration 30s
Artifacts –

checks.yml

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

Annotations

17 warnings and 12 notices
fixme: scripts/metricsLib/oss_metric_entities.py#L121
TODO: should this logic be moved to the hit_metric method?
broad-exception-caught: scripts/metricsLib/oss_metric_entities.py#L227
Catching too general exception Exception
broad-exception-caught: scripts/metricsLib/oss_metric_entities.py#L352
Catching too general exception Exception
fixme: scripts/metricsLib/metrics_definitions.py#L139
TODO: Ask Sean why this endpoint isn't working for any repo except for augur.
too-many-arguments: scripts/metricsLib/metrics_data_structures.py#L51
Too many arguments (7/5)
too-many-arguments: scripts/metricsLib/metrics_data_structures.py#L140
Too many arguments (6/5)
arguments-renamed: scripts/metricsLib/metrics_data_structures.py#L172
Parameter 'params' has been renamed to 'oss_entity' in overriding 'ResourceMetric.get_values' method
too-many-arguments: scripts/metricsLib/metrics_data_structures.py#L211
Too many arguments (7/5)
no-else-raise: scripts/metricsLib/metrics_data_structures.py#L247
Unnecessary "else" after "raise", remove the "else" and de-indent the code inside it
too-many-arguments: scripts/metricsLib/metrics_data_structures.py#L277
Too many arguments (7/5)
too-many-arguments: scripts/metricsLib/metrics_data_structures.py#L298
Too many arguments (7/5)
too-many-arguments: scripts/metricsLib/metrics_data_structures.py#L347
Too many arguments (7/5)
too-many-arguments: scripts/metricsLib/metrics_data_structures.py#L392
Too many arguments (7/5)
fixme: scripts/refresh_metrics.py#L43
TODO: Create a read repos-to-include.txt
fixme: scripts/gen_graphs.py#L66
TODO: file a pygals issue to make sparklines their own object
fixme: scripts/gen_graphs.py#L127
TODO: Just get these metrics from augur instead of storing them in json.
dangerous-default-value: scripts/gen_graphs.py#L27
Dangerous default value {} as argument
invalid-name: scripts/metricsLib/__init__.py#L1
Module name "metricsLib" doesn't conform to snake_case naming style
line-too-long: scripts/gen_graphs.py#L73
Line too long (106/100)
missing-function-docstring: scripts/gen_graphs.py#L21
Missing function or method docstring
consider-using-f-string: scripts/gen_graphs.py#L87
Formatting a regular string which could be an f-string
missing-function-docstring: scripts/gen_reports.py#L33
Missing function or method docstring
missing-function-docstring: scripts/gen_reports.py#L66
Missing function or method docstring
missing-function-docstring: scripts/gen_reports.py#L72
Missing function or method docstring
missing-module-docstring: scripts/run_pylint.py#L1
Missing module docstring
line-too-long: scripts/tests/metrics_tests.py#L14
Line too long (141/100)
trailing-whitespace: scripts/tests/metrics_tests.py#L48
Trailing whitespace
missing-function-docstring: scripts/tests/metrics_tests.py#L7
Missing function or method docstring
missing-function-docstring: scripts/tests/metrics_tests.py#L11
Missing function or method docstring