Skip to content

Commit 8085db4

Browse files
committed
Clean up lint findings exposed by black 26 reformat
- Ignore E704 in flake8: black 26 collapses abstract method stubs (def foo(): ...) onto one line; E704 conflicts with that style. - Drop unused imports RecommendationValue and NAN_LITERAL in formatters/table.py. - Replace unnecessary f-string with plain triple-quoted string in strategies/simple.py (F541).
1 parent 07fb42f commit 8085db4

3 files changed

Lines changed: 2 additions & 4 deletions

File tree

.flake8

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -12,4 +12,4 @@ exclude = .git,
1212
.mypy_cache,
1313
src/robusta/integrations/kubernetes/autogenerated,
1414
src/robusta/integrations/kubernetes/custom_models.py
15-
ignore = E501, W503, E203
15+
ignore = E501, W503, E203, E704

robusta_krr/formatters/table.py

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -5,11 +5,9 @@
55

66
from robusta_krr.core.abstract import formatters
77
from robusta_krr.core.models.allocations import (
8-
RecommendationValue,
98
format_recommendation_value,
109
format_diff,
1110
NONE_LITERAL,
12-
NAN_LITERAL,
1311
)
1412
from robusta_krr.core.models.result import ResourceScan, ResourceType, Result
1513
from robusta_krr.core.models.config import settings

robusta_krr/strategies/simple.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -101,7 +101,7 @@ def description(self):
101101
""")
102102

103103
if not self.settings.allow_hpa:
104-
s += "\n" + textwrap.dedent(f"""\
104+
s += "\n" + textwrap.dedent("""\
105105
This strategy does not work with objects with HPA defined (Horizontal Pod Autoscaler).
106106
If HPA is defined for CPU or Memory, the strategy will return "?" for that resource.
107107
You can override this behaviour by passing the --allow-hpa flag

0 commit comments

Comments
 (0)