add ruff python linting

pantor · Aug 17, 2023 · 993a537 · 993a537
1 parent 2737109
commit 993a537
Show file tree

Hide file tree

Showing 7 changed files with 27 additions and 7 deletions.
diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
@@ -74,3 +74,15 @@ jobs:
       if: matrix.os == 'ubuntu-latest'
       run: |
         env CTEST_OUTPUT_ON_FAILURE=1 ./build/test-target 5000000
+
+
+  lint-python:
+    runs-on: ubuntu-latest
+
+    steps:
+    - uses: actions/checkout@v3
+
+    - name: Lint Python
+      run: |
+        python3 -m pip install ruff
+        ruff examples test
diff --git a/examples/03_waypoints.py b/examples/03_waypoints.py
@@ -25,7 +25,7 @@
         [1.4, -1.6, 1.0],
         [-0.6, -0.5, 0.4],
         [-0.4, -0.35, 0.0],
-        [0.8, 1.8, -0.1]
+        [0.8, 1.8, -0.1],
     ]
 
     inp.target_position = [0.5, 1, 0]

diff --git a/examples/04_waypoints_online.py b/examples/04_waypoints_online.py
@@ -25,7 +25,7 @@
         [1.4, -1.6, 1.0],
         [-0.6, -0.5, 0.4],
         [-0.4, -0.35, 0.0],
-        [0.8, 1.8, -0.1]
+        [0.8, 1.8, -0.1],
     ]
 
     inp.target_position = [0.5, 1, 0]

diff --git a/examples/08_per_section_minimum_duration.py b/examples/08_per_section_minimum_duration.py
@@ -27,7 +27,7 @@
         [-0.4, -0.35, 0.0],
         [-0.2, 0.35, -0.1],
         [0.2, 0.5, -0.1],
-        [0.8, 1.8, -0.1]
+        [0.8, 1.8, -0.1],
     ]
 
     inp.target_position = [0.5, 1.2, 0]

diff --git a/pyproject.toml b/pyproject.toml
@@ -1,3 +1,12 @@
 [build-system]
 requires = ["setuptools>=42", "wheel", "pybind11~=2.6.1", "ninja", "cmake>=3.10"]
-build-backend = "setuptools.build_meta"
+build-backend = "setuptools.build_meta"
+
+[tool.ruff]
+select = ["A", "COM", "E", "F", "G", "N", "PIE", "PTH", "PYI", "RSE", "RET", "SIM", "TCH", "W", "Q", "E201", "E202", "E203"]
+line-length = 160
+ignore = ["E402"]
+
+[tool.ruff.flake8-quotes]
+inline-quotes = "single"
+multiline-quotes = "single"
diff --git a/test/plot_target.py b/test/plot_target.py
@@ -6,7 +6,7 @@
 
 path.insert(0, str(Path(__file__).parent.absolute().parent / 'build'))
 
-from ruckig import InputParameter, OutputParameter, Result, Ruckig, Synchronization, ControlInterface, DurationDiscretization
+from ruckig import InputParameter, OutputParameter, Result, Ruckig, Synchronization, ControlInterface, DurationDiscretization  # noqa: F401
 
 
 def walk_through_trajectory(otg, inp):

diff --git a/test/plot_waypoints.py b/test/plot_waypoints.py
@@ -3,11 +3,10 @@
 from sys import path
 
 from plotter import Plotter
-import numpy as np
 
 path.insert(0, str(Path(__file__).parent.absolute().parent / 'build'))
 
-from ruckig import InputParameter, OutputParameter, Result, Ruckig, Trajectory, ControlInterface
+from ruckig import InputParameter, OutputParameter, Result, Ruckig, ControlInterface  # noqa: F401
 
 
 def walk_through_trajectory(otg, inp):