Back to pull request #505

#504: verified the new logic to test and count all constraints indivi… #779

Sign in to view logs

Triggered via push April 23, 2024 15:01

ppebay

504-Make-FWMP-constraint-verificator-part-of-LBAF-scripts

Status Cancelled

Total duration 3h 30m 25s

Artifacts –

code-quality.yml

on: push

Matrix: code-quality

Annotations

2 errors, 11 warnings, and 10 notices

code-quality (ubuntu-latest, 3.8)

Canceling since a higher priority waiting request for 'CI-' exists

code-quality (ubuntu-latest, 3.8)

The operation was canceled.

code-quality (ubuntu-latest, 3.9)

Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: actions/checkout@v3, actions/setup-python@v4, phoenix-actions/test-reporting@v12. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.

code-quality (ubuntu-latest, 3.8): src/lbaf/Utils/lbsJSONDataFilesValidatorLoader.py#L10

Unused PROJECT_PATH imported from lbaf (unused-import)

code-quality (ubuntu-latest, 3.8): src/lbaf/Model/lbsPhase.py#L464

Dangerous default value [] as argument (dangerous-default-value)

code-quality (ubuntu-latest, 3.8): src/lbaf/Model/lbsPhase.py#L44

Unused private member `Phase.__metadata` (unused-private-member)

code-quality (ubuntu-latest, 3.8): src/lbaf/Model/lbsRank.py#L187

Unused argument 'r_dst' (unused-argument)

code-quality (ubuntu-latest, 3.8): src/lbaf/Model/lbsRank.py#L20

Unused private member `Rank.__logger` (unused-private-member)

code-quality (ubuntu-latest, 3.8): src/lbaf/Execution/lbsClusteringTransferStrategy.py#L82

Unused variable 'i' (unused-variable)

code-quality (ubuntu-latest, 3.8): src/lbaf/Execution/lbsClusteringTransferStrategy.py#L198

Parameter 'phase' has been renamed to 'known_peers' in overridden 'ClusteringTransferStrategy.execute' method (arguments-renamed)

code-quality (ubuntu-latest, 3.8): src/lbaf/Execution/lbsClusteringTransferStrategy.py#L198

Parameter 'known_peers' has been renamed to 'phase' in overridden 'ClusteringTransferStrategy.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/lbsCentralizedPrefixOptimizerAlgorithm.py#L152

Lambda may not be necessary (unnecessary-lambda)

code-quality (ubuntu-latest, 3.8): src/lbaf/Utils/lbsDataStatFilesUpdater.py#L18

Too few public methods (1/2) (too-few-public-methods)

code-quality (ubuntu-latest, 3.8): src/lbaf/Utils/lbsPath.py#L19

Unnecessary "else" after "return" (no-else-return)

code-quality (ubuntu-latest, 3.8): src/lbaf/Utils/lbsJSONDataFilesValidatorLoader.py#L1

Missing module docstring (missing-module-docstring)

code-quality (ubuntu-latest, 3.8): src/lbaf/Utils/lbsJSONDataFilesValidatorLoader.py#L66

Method could be a function (no-self-use)

code-quality (ubuntu-latest, 3.8): src/lbaf/Utils/lbsArgumentParser.py#L18

Too many arguments (14/7) (too-many-arguments)

code-quality (ubuntu-latest, 3.8): src/lbaf/Utils/lbsArgumentParser.py#L36

Consider using Python 3 style super() without arguments (super-with-arguments)

code-quality (ubuntu-latest, 3.8): src/lbaf/Utils/lbsArgumentParser.py#L82

Consider merging these comparisons with "in" to "raw_response in ('', 'None')" (consider-using-in)

code-quality (ubuntu-latest, 3.8): src/lbaf/Utils/lbsArgumentParser.py#L58

Method could be a function (no-self-use)

code-quality (ubuntu-latest, 3.8): src/lbaf/Utils/lbsArgumentParser.py#L58

Too many branches (17/12) (too-many-branches)

code-quality (ubuntu-latest, 3.8): src/lbaf/Utils/lbsArgumentParser.py#L149

Unnecessary "else" after "return" (no-else-return)