Merge pull request #496 from riscv-software-src/pylint_workflow

debug: Only run pylint if debug files changed.

2 files changed, 9 insertions, 5 deletions

diff --git a/.github/workflows/debug.yml b/.github/workflows/debug.yml
index 29ca5b2..7b6beb4 100644
--- a/.github/workflows/debug.yml
+++ b/.github/workflows/debug.yml
@@ -1,6 +1,9 @@
-on: pull_request
+on:
+  pull_request:
+    paths:
+    - 'debug/**'
 
-name: Check Code Style (checkpatch)
+name: Check Debug Code Style (pylint)
 
 jobs:
   check:
diff --git a/debug/gdbserver.py b/debug/gdbserver.py
index 53eeffa..3b9a661 100755
--- a/debug/gdbserver.py
+++ b/debug/gdbserver.py
@@ -1562,6 +1562,7 @@ class DownloadTest(GdbTest):
 #        assertIn("0xbead", output)
 
 class PrivTest(GdbSingleHartTest):
+    """Base class for a few tests that change privilege levels."""
     compile_args = ("programs/priv.S", )
     def setup(self):
         # pylint: disable=attribute-defined-outside-init
@@ -1588,8 +1589,8 @@ class PrivTest(GdbSingleHartTest):
             pass
 
 class PrivRw(PrivTest):
+    """Test reading/writing priv."""
     def test(self):
-        """Test reading/writing priv."""
         self.write_nop_program(4)
         for privilege in range(4):
             self.gdb.p(f"$priv={privilege}")
@@ -1600,9 +1601,9 @@ class PrivRw(PrivTest):
                 assertEqual(actual, privilege)
 
 class PrivChange(PrivTest):
+    """Test that the core's privilege level actually changes when the debugger
+    writes it."""
     def test(self):
-        """Test that the core's privilege level actually changes."""
-
         if 0 not in self.supported:
             raise TestNotApplicable