Patch API has a method returning a no-result method call
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
StarlingX |
Fix Released
|
Low
|
Don Penney |
Bug Description
Brief Description
-----------------
Pylint for cgcs-patch currently has an error code suppressed:
E1111:(
This code should be un-supressed and the area that is raising this error should be fixed, as it means that the code is always returning a blank result.
cgcs_patch/
that method is calling a routine that does not return a result: https:/
Severity
--------
Minor
Steps to Reproduce
------------------
1) Remove E1111 from pylint.rc in cgcs_patch and run 'tox -e pylint'
The following steps are optional:
2) - open google chrome.
3) - do a search for amusing cat videos
4) - watch amusing cat videos
5) - then run step 1
Expected Behavior
------------------
pylint should run clean once that error code is fixed.
Actual Behavior
----------------
Currently will fail (because the code needs to be fixed)
Reproducibility
---------------
Its a code issue tied to a static linting tool. It can be reproduced 100% of the time.
System Configuration
-------
N/A
Branch/Pull Time/Commit
-------
Any time after Dec 4th in master branch of starlingx/update
Error is detectable with the newer version of pylint which is introduced by
https:/
Last Pass
---------
N/A
Timestamp/Logs
--------------
N/A
Test Activity
-------------
Developer Development.
Changed in starlingx: | |
assignee: | nobody → Don Penney (dpenney) |
Changed in starlingx: | |
importance: | Undecided → Low |
Fix proposed to branch: master /review. opendev. org/697408
Review: https:/