Merge pull request #258 from opsmill/develop

Release v1.7.1

Author: lykinsbd

.github/workflows/ci.yml

@@ -151,8 +151,6 @@ jobs:
         run: "poetry install --all-extras"
       - name: "Mypy Tests"
         run: "poetry run mypy --show-error-codes infrahub_sdk/"
-      # - name: "Pylint Tests"
-      #   run: "poetry run pylint infrahub_sdk/"
       - name: "Unit Tests"
         run: "poetry run pytest --cov infrahub_sdk tests/unit/"
       - name: "Upload coverage to Codecov"

CHANGELOG.md

@@ -11,6 +11,12 @@ This project uses [*towncrier*](https://towncrier.readthedocs.io/) and the chang
 
 <!-- towncrier release notes start -->
 
+## [1.7.1](https://github.com/opsmill/infrahub-sdk-python/tree/v1.7.1) - 2025-01-30
+
+### Removed
+
+- All mention of pylint have been removed from the project. ([#206](https://github.com/opsmill/infrahub-sdk-python/issues/206))
+
 ## [1.7.0](https://github.com/opsmill/infrahub-sdk-python/tree/v1.7.0) - 2025-01-23
 
 ### Added

infrahub_sdk/batch.py

@@ -30,7 +30,7 @@ def execute(self, return_exceptions: bool = False) -> tuple[InfrahubNodeSync | N
         result = None
         try:
             result = self.task(*self.args, **self.kwargs)
-        except Exception as exc:  # pylint: disable=broad-exception-caught
+        except Exception as exc:
             if return_exceptions:
                 return self.node, exc
             raise exc
@@ -44,7 +44,7 @@ async def execute_batch_task_in_pool(
     async with semaphore:
         try:
             result = await task.task(*task.args, **task.kwargs)
-        except Exception as exc:  # pylint: disable=broad-exception-caught
+        except Exception as exc:
             if return_exceptions:
                 return (task.node, exc)
             raise exc

infrahub_sdk/config.py

@@ -113,7 +113,7 @@ def validate_address(cls, value: str) -> str:
 
     @model_validator(mode="after")
     def validate_proxy_config(self) -> Self:
-        if self.proxy and self.proxy_mounts.is_set:  # pylint: disable=no-member
+        if self.proxy and self.proxy_mounts.is_set:
             raise ValueError("'proxy' and 'proxy_mounts' are mutually exclusive")
         return self
 

infrahub_sdk/ctl/check.py

@@ -121,7 +121,7 @@ async def run_check(
     except QueryNotFoundError as exc:
         log.warning(f"{module_name}::{check}: unable to find query ({exc!s})")
         passed = False
-    except Exception as exc:  # pylint: disable=broad-exception-caught
+    except Exception as exc:
         log.warning(f"{module_name}::{check}: An error occurred during execution ({exc})")
         passed = False
 

infrahub_sdk/ctl/utils.py

@@ -88,7 +88,7 @@ def decorator(func: Callable[..., T]) -> Callable[..., T | Coroutine[Any, Any, T
             async def async_wrapper(*args: Any, **kwargs: Any) -> T:
                 try:
                     return await func(*args, **kwargs)
-                except (Error, Exception) as exc:  # pylint: disable=broad-exception-caught
+                except (Error, Exception) as exc:
                     return handle_exception(exc=exc, console=console, exit_code=exit_code)
 
             return async_wrapper
@@ -97,7 +97,7 @@ async def async_wrapper(*args: Any, **kwargs: Any) -> T:
         def wrapper(*args: Any, **kwargs: Any) -> T:
             try:
                 return func(*args, **kwargs)
-            except (Error, Exception) as exc:  # pylint: disable=broad-exception-caught
+            except (Error, Exception) as exc:
                 return handle_exception(exc=exc, console=console, exit_code=exit_code)
 
         return wrapper

infrahub_sdk/data.py

@@ -20,7 +20,7 @@ class RepositoryData(BaseModel):
     branch_info: dict[str, RepositoryBranchInfo] = Field(default_factory=dict)
 
     def get_staging_branch(self) -> str | None:
-        for branch, info in self.branch_info.items():  # pylint: disable=no-member
+        for branch, info in self.branch_info.items():
             if info.internal_status == "staging":
                 return branch
         return None

infrahub_sdk/node.py

@@ -25,7 +25,6 @@
     from .schema import AttributeSchemaAPI, MainSchemaTypesAPI, RelationshipSchemaAPI
     from .types import Order
 
-# pylint: disable=too-many-lines
 
 PROPERTIES_FLAG = ["is_visible", "is_protected"]
 PROPERTIES_OBJECT = ["source", "owner"]
@@ -801,7 +800,7 @@ def _generate_input_data(self, exclude_unmodified: bool = False, exclude_hfid: b
         Returns:
             dict[str, Dict]: Representation of an input data in dict format
         """
-        # pylint: disable=too-many-branches
+
         data = {}
         variables = {}
 
@@ -1251,7 +1250,6 @@ async def generate_query_data_node(
         Returns:
             dict[str, Union[Any, Dict]]: GraphQL query in dictionary format
         """
-        # pylint: disable=too-many-branches
 
         data: dict[str, Any] = {}
 
@@ -1763,7 +1761,6 @@ def generate_query_data_node(
         Returns:
             dict[str, Union[Any, Dict]]: GraphQL query in dictionary format
         """
-        # pylint: disable=too-many-branches
 
         data: dict[str, Any] = {}
 

infrahub_sdk/protocols.py

@@ -29,7 +29,6 @@
         StringOptional,
     )
 
-# pylint: disable=too-many-ancestors
 
 # ---------------------------------------------
 # ASYNC

infrahub_sdk/schema/__init__.py

@@ -61,9 +61,6 @@
 ]
 
 
-# pylint: disable=redefined-builtin
-
-
 class DropdownMutationOptionalArgs(TypedDict):
     color: str | None
     description: str | None