Use ruff to lint/format/isort

Makefile

@@ -2,14 +2,14 @@ clean:
 	find . -type d -name '__pycache__' | xargs rm -rf
 
 _lint:
-	flake8 --exclude=**/gen/**,*_grpc.py,*_pb2.py,*_pb2.pyi,.tox
+	ruff --exclude=*_grpc.py,*_pb2.py,*_pb2.pyi,.tox .
 
 lint:
 	poetry run $(MAKE) _lint
 
 _format:
-	black --exclude ".*/gen/.*" ./src
-	isort ./src
+	ruff format ./src
+	ruff --select I001 --fix ./src
 
 format:
 	poetry run $(MAKE) _format

pyproject.toml

@@ -39,6 +39,7 @@ include = ["LICENSE", "src/viam/rpc/libviam_rust_utils.*"]
   pytest-watcher = "^0.3.4"
   # TODO(RSDK-5450) - Update myst-nb so this can be unpinned.
   astroid = "<=2.15.6"
+  ruff = "^0.1.3"
 
 [tool.pytest.ini_options]
 addopts = "-ra"
@@ -51,17 +52,17 @@ omit = [ "*/gen/*" ]
 [tool.coverage.report]
 exclude_lines = [ "pragma: no\\s*cover", "\\.\\.\\." ]
 
-[tool.black]
-line-length = 140
-
-[tool.isort]
-profile = "black"
-skip_glob = ["**/gen/**"]
-line_length = 140
-
 [build-system]
 requires = [ "poetry-core>=1.0.0" ]
 build-backend = "poetry.core.masonry.api"
 
 [tool.poetry.extras]
 mlmodel = ["numpy"]
+
+[tool.ruff]
+line-length = 140
+ignore = ["E203"]
+extend-exclude = ["**/gen/**"]
+
+[tool.ruff.extend-per-file-ignores]
+"__init__.py" = ["F401"]

src/viam/app/app_client.py

@@ -28,9 +28,6 @@
     DeleteRobotPartRequest,
     DeleteRobotPartSecretRequest,
     DeleteRobotRequest,
-)
-from viam.proto.app import Fragment as FragmentPB
-from viam.proto.app import (
     GetFragmentRequest,
     GetFragmentResponse,
     GetLocationRequest,
@@ -67,9 +64,6 @@
     LocationAuth,
     LocationAuthRequest,
     LocationAuthResponse,
-)
-from viam.proto.app import LogEntry as LogEntryPB
-from viam.proto.app import (
     MarkPartAsMainRequest,
     MarkPartForRestartRequest,
     Model,
@@ -85,10 +79,6 @@
     OrgDetails,
     RemoveRoleRequest,
     Robot,
-)
-from viam.proto.app import RobotPart as RobotPartPB
-from viam.proto.app import RobotPartHistoryEntry as RobotPartHistoryEntryPB
-from viam.proto.app import (
     SharedSecret,
     TailRobotPartLogsRequest,
     TailRobotPartLogsResponse,
@@ -107,6 +97,10 @@
     UploadModuleFileRequest,
     Visibility,
 )
+from viam.proto.app import Fragment as FragmentPB
+from viam.proto.app import LogEntry as LogEntryPB
+from viam.proto.app import RobotPart as RobotPartPB
+from viam.proto.app import RobotPartHistoryEntry as RobotPartHistoryEntryPB
 from viam.utils import datetime_to_timestamp, dict_to_struct, struct_to_dict
 
 LOGGER = logging.getLogger(__name__)

src/viam/resource/registry.py

@@ -33,7 +33,7 @@ class ResourceCreatorRegistration:
     """A function that can create a resource given a mapping of dependencies (``ResourceName`` to ``ResourceBase``
     """
 
-    validator: "Validator" = lambda x: []
+    validator: "Validator" = lambda x: []  # noqa: E731
     """A function that can validate a resource and return implicit dependencies.
 
     If called without a validator function, default to a function returning an empty Sequence