Merge pull request #119 from desy-ml/118-failing-pipeline-due-to-new-flake8-bugbear-version

jank324 · web-flow · commit d9ddf034caf2 · 2024-01-16T15:52:42.000+01:00
Ignore iterable modification flake8 warning
diff --git a/.flake8 b/.flake8
@@ -7,4 +7,4 @@ exclude =
     .ipynb_checkpoints
 max-line-length = 88
 extend-ignore = E203, W503
-ignore = BLK100
+ignore = BLK100
diff --git a/cheetah/converters/dontbmad.py b/cheetah/converters/dontbmad.py
@@ -426,7 +426,7 @@ def validate_understood_properties(understood: list[str], properties: dict) -> N
         assert property in understood, (
             f"Property {property} with value {properties[property]} for element type"
             f" {properties['element_type']} is currently not understood. Other values"
-            f" in properties are {properties.keys()}."
+            f" in properties are {properties.keys()}."  # noqa: B038
         )
 
 

Original file line number	Diff line number	Diff line change
`@@ -426,7 +426,7 @@ def validate_understood_properties(understood: list[str], properties: dict) -> N`
`426`	`426`	`assert property in understood, (`
`427`	`427`	`f"Property {property} with value {properties[property]} for element type"`
`428`	`428`	`f" {properties['element_type']} is currently not understood. Other values"`
`429`		`- f" in properties are {properties.keys()}."`
	`429`	`+ f" in properties are {properties.keys()}." # noqa: B038`
`430`	`430`	`)`
`431`	`431`
`432`	`432`