requirements-dev: use black 23.12.1 and ruff 0.1.9

Fix ruff's recommendation: phase2/master.cfg:328:16: E721 Do not compare types, use `isinstance()` "Unlike a direct type comparison, isinstance will also check if an object is an instance of a class or a subclass thereof." Signed-off-by: Petr Štetiar <[email protected]>
openwrt · Dec 24, 2023 · e8bfef1 · e8bfef1
1 parent 03abd02
commit e8bfef1
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 4 deletions.
diff --git a/phase1/master.cfg b/phase1/master.cfg
@@ -266,7 +266,8 @@ def prioritizeBuilders(master, builders):
         pos = 99
         for name, prio in bldrNamePrio.items():
             if bldr.name.startswith(name):
-                pos = prio + 50 - min(hiprio, 50)    # higher priority (larger positive number) raises position
+                # higher priority (larger positive number) raises position
+                pos = prio + 50 - min(hiprio, 50)
                 break
 
         # pos order: janitor/local (0), tag builds if any [1..50], !tag builds [51...]

diff --git a/phase2/master.cfg b/phase2/master.cfg
@@ -280,7 +280,7 @@ def IsArchitectureSelected(target):
 	def CheckArchitectureProperty(step):
 		try:
 			options = step.getProperty("options")
-			if type(options) is dict:
+			if isinstance(options, dict):
 				selected_arch = options.get("architecture", "all")
 				if selected_arch != "all" and selected_arch != target:
 					return False

diff --git a/requirements-dev.txt b/requirements-dev.txt
@@ -1,4 +1,4 @@
 cram==0.7
-black==23.3.0
-ruff==0.0.267
+black==23.12.1
+ruff==0.1.9
 flake8==6.1.0