diff options
| author |   Vratko Polak <vrpolak@cisco.com> | 2019-12-04 13:24:07 +0100 |
|---|---|---|
| committer |   Peter Mikus <pmikus@cisco.com> | 2019-12-05 07:03:57 +0000 |
| commit | 063abf35e81deaf749ebbcfee339fbd1d9e89412 (patch) | |
| tree | c932eac04f162c46b0f3c38db10105b945a617cb /resources/libraries/python/GBP.py | |
| parent | 6221f1b96d2a167c6db74ff26cd7ec7906ae9486 (diff) | |
Deal with some "pylint: disable=" comments
+ When possible, fix the violation.
+ Else, add a comment:
+ An explanation (if not already present) and keep disable.
+ A TODO (if not already present) and remove the disable.
- This makes tox job report more pylint violations,
but any such violation is fixable and should be fixed.
- Although some need to be fixed in VPP, such as enum item long names.
Change-Id: I48604b5eda070083d79dff1439620dbd9e798e1f
Signed-off-by: Vratko Polak <vrpolak@cisco.com>
Diffstat (limited to 'resources/libraries/python/GBP.py')
| -rw-r--r-- | resources/libraries/python/GBP.py | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/resources/libraries/python/GBP.py b/resources/libraries/python/GBP.py index 3d4d249b10..59dd269d95 100644 --- a/resources/libraries/python/GBP.py +++ b/resources/libraries/python/GBP.py @@ -43,8 +43,9 @@ class GBPBridgeDomainFlags(IntEnum): class GBPSubnetType(IntEnum): """GBP Subnet Type.""" GBP_API_SUBNET_TRANSPORT = 1 - GBP_API_SUBNET_STITCHED_INTERNAL = 2 # pylint: disable=invalid-name - GBP_API_SUBNET_STITCHED_EXTERNAL = 3 # pylint: disable=invalid-name + # TODO: Names too long for pylint, fix in VPP. + GBP_API_SUBNET_STITCHED_INTERNAL = 2 + GBP_API_SUBNET_STITCHED_EXTERNAL = 3 GBP_API_SUBNET_L3_OUT = 4 GBP_API_SUBNET_ANON_L3_OUT = 5 |