Run ruff in CI

testing-cabal · Nov 24, 2023 · 3d65835 · 3d65835
1 parent 98aa107
commit 3d65835
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 2 deletions.
diff --git a/.github/workflows/main.yml b/.github/workflows/main.yml
@@ -21,9 +21,13 @@ jobs:
       - name: Install Deps
         run: sudo apt-get install check libcppunit-dev
       - name: Install package
-        run: python -m pip install -U '.[test,docs]'
+        run: |
+          python -m pip install ruff
+          python -m pip install -U '.[test,docs]'
       - name: Build
         run: autoreconf -fi && ./configure && make
+      - name: Run ruff check
+        run: ruff check python
       - name: Run make check
         run: make check
       - name: Run make distcheck

diff --git a/python/subunit/__init__.py b/python/subunit/__init__.py
@@ -762,7 +762,7 @@ def addUnexpectedSuccess(self, test, details=None):
 
     def _test_id(self, test):
         result = test.id()
-        if type(result) is not bytes:
+        if not isinstance(result, bytes):
             result = result.encode('utf8')
         return result