diff --git a/base/images/tests/utils/pytest_plugin.py b/base/images/tests/utils/pytest_plugin.py index ee95e4403fc..e096a32c815 100644 --- a/base/images/tests/utils/pytest_plugin.py +++ b/base/images/tests/utils/pytest_plugin.py @@ -225,6 +225,7 @@ def pytest_collection_modifyitems( Tests directly under ``cases/static/`` or ``cases/runtime/`` (no image subdir) get no ``image`` marker and run for every image. """ + del config from pathlib import Path for item in items: diff --git a/ruff.toml b/ruff.toml index da01a75bbb9..83cb538f462 100644 --- a/ruff.toml +++ b/ruff.toml @@ -19,9 +19,6 @@ select = ["ALL"] # Everything ignore = [ # Existing repository debt. Remove codes from this baseline as focused # cleanup changes make each rule green. - "ARG001", - "ARG002", - "B007", "B904", "BLE001", "C901", diff --git a/scripts/ci/control-tower/tests/test_package_build.py b/scripts/ci/control-tower/tests/test_package_build.py index aa3e0191497..27fe615cbfe 100644 --- a/scripts/ci/control-tower/tests/test_package_build.py +++ b/scripts/ci/control-tower/tests/test_package_build.py @@ -49,9 +49,9 @@ def _submit(*, wait_for_completion: bool = False) -> None: class TestPackageBuildLifecycle: """Verify shared submission and polling behavior.""" + @pytest.mark.usefixtures("lifecycle_mocks") def test_uses_pipeline_service_connection( self, - lifecycle_mocks: tuple[Mock, Mock, Mock, Mock], monkeypatch: pytest.MonkeyPatch, ) -> None: """Use the identity authenticated by the enclosing AzureCLI task.""" diff --git a/scripts/repo/synthesize-repodata.py b/scripts/repo/synthesize-repodata.py index 827980f5b11..351965b8152 100755 --- a/scripts/repo/synthesize-repodata.py +++ b/scripts/repo/synthesize-repodata.py @@ -716,7 +716,7 @@ def decide_routing( """ decisions: dict[UniverseKey, RoutingDecision] = {} tied_components_warned: set[str] = set() - for key, entry in universe.items(): + for key in universe: kind = key[0] name = key[2] # Foreign packages (azldev fell back to project defaults for an @@ -1066,6 +1066,8 @@ def __call__( values: str | Sequence[object] | None, option_string: str | None = None, ) -> None: + del parser + items = getattr(namespace, self.dest, None) if items is None: items = []