Skip to content

#469: remove extraneous code #623

#469: remove extraneous code

#469: remove extraneous code #623

Triggered via push October 18, 2023 19:44
Status Success
Total duration 3m 56s
Artifacts 2
This run and associated checks have been archived and are scheduled for deletion. Learn more about checks retention

code-quality.yml

on: push
Matrix: code-quality
Fit to window
Zoom out
Zoom in

Annotations

10 warnings and 10 notices
code-quality (ubuntu-latest, 3.8): src/lbaf/Execution/lbsAlgorithmBase.py#L250
Unnecessary pass statement (unnecessary-pass)
code-quality (ubuntu-latest, 3.8): src/lbaf/Execution/lbsCentralizedPrefixOptimizerAlgorithm.py#L152
Lambda may not be necessary (unnecessary-lambda)
code-quality (ubuntu-latest, 3.8): src/lbaf/Execution/lbsRecursiveTransferStrategy.py#L63
Parameter 'phase' has been renamed to 'known_peers' in overridden 'RecursiveTransferStrategy.execute' method (arguments-renamed)
code-quality (ubuntu-latest, 3.8): src/lbaf/Execution/lbsRecursiveTransferStrategy.py#L63
Parameter 'known_peers' has been renamed to 'phase' in overridden 'RecursiveTransferStrategy.execute' method (arguments-renamed)
code-quality (ubuntu-latest, 3.8): src/lbaf/Execution/lbsTemperedCriterion.py#L8
Method 'estimate' is abstract in class 'CriterionBase' but is not overridden (abstract-method)
code-quality (ubuntu-latest, 3.8): src/lbaf/Execution/lbsCriterionBase.py#L66
Dangerous default value [] as argument (dangerous-default-value)
code-quality (ubuntu-latest, 3.8): src/lbaf/Execution/lbsCriterionBase.py#L76
Unnecessary pass statement (unnecessary-pass)
code-quality (ubuntu-latest, 3.8): src/lbaf/Execution/lbsCriterionBase.py#L79
Dangerous default value [] as argument (dangerous-default-value)
code-quality (ubuntu-latest, 3.8): src/lbaf/Execution/lbsClusteringTransferStrategy.py#L92
Parameter 'phase' has been renamed to 'known_peers' in overridden 'ClusteringTransferStrategy.execute' method (arguments-renamed)
code-quality (ubuntu-latest, 3.8): src/lbaf/IO/lbsVTDataReader.py#L34
Line too long (124/120) (line-too-long)
code-quality (ubuntu-latest, 3.8): src/lbaf/IO/lbsVTDataReader.py#L1
Missing module docstring (missing-module-docstring)
code-quality (ubuntu-latest, 3.8): src/lbaf/IO/lbsVTDataReader.py#L52
Import outside toplevel (imported.JSON_data_files_validator.SchemaValidator) (import-outside-toplevel)
code-quality (ubuntu-latest, 3.8): src/lbaf/IO/lbsVTDataReader.py#L77
Variable name "l" doesn't conform to snake_case naming style (invalid-name)
code-quality (ubuntu-latest, 3.8): src/lbaf/IO/lbsVTDataReader.py#L148
Too many local variables (33/15) (too-many-locals)
code-quality (ubuntu-latest, 3.8): src/lbaf/IO/lbsVTDataReader.py#L154
Unnecessary "else" after "continue" (no-else-continue)
code-quality (ubuntu-latest, 3.8): src/lbaf/IO/lbsVTDataReader.py#L148
Too many branches (15/12) (too-many-branches)
code-quality (ubuntu-latest, 3.8): src/lbaf/IO/lbsVTDataReader.py#L269
Too many local variables (18/15) (too-many-locals)
code-quality (ubuntu-latest, 3.8): src/lbaf/IO/lbsVTDataReader.py#L284
Variable name "c" doesn't conform to snake_case naming style (invalid-name)
code-quality (ubuntu-latest, 3.8): src/lbaf/IO/lbsVTDataReader.py#L297
Variable name "r" doesn't conform to snake_case naming style (invalid-name)

Artifacts

Produced during runtime
Name Size
coverage Expired
5.95 KB
pylint Expired
38.4 KB