Merge pull request #4790 from opsmill/pog-furb116

Fix FURB116 Replace `bin` call with f-string

Author: ogenstad

backend/infrahub/core/utils.py

@@ -184,10 +184,10 @@ def convert_ip_to_binary_str(
     obj: Union[ipaddress.IPv6Network, ipaddress.IPv4Network, ipaddress.IPv4Interface, ipaddress.IPv6Interface],
 ) -> str:
     if isinstance(obj, (ipaddress.IPv6Network, ipaddress.IPv4Network)):
-        prefix_bin = bin(int(obj.network_address))[2:]
+        prefix_bin = f"{int(obj.network_address):b}"
         return prefix_bin.zfill(obj.max_prefixlen)
 
-    ip_bin = bin(int(obj))[2:]
+    ip_bin = f"{int(obj):b}"
     return ip_bin.zfill(obj.max_prefixlen)
 
 

pyproject.toml

@@ -489,7 +489,6 @@ ignore = [
     "C414",     # Unnecessary `list` call within `sorted()`
     "C420",     # Unnecessary dict comprehension for iterable; use `dict.fromkeys` instead
     "FURB113",  # Use `networks.extend(...)` instead of repeatedly calling `networks.append()`
-    "FURB116",  # Replace `bin` call with f-string
     "FURB118",  # Use `operator.itemgetter(1)` instead of defining a lambda
     "FURB140",  # Use `itertools.starmap` instead of the generator
     "FURB171",  # Membership test against single-item container