test workflows

.github/workflows/pr.yml

@@ -12,20 +12,20 @@ concurrency:
   cancel-in-progress: true
 
 jobs:
-  changes:
-    runs-on: ubuntu-latest
-    permissions:
-      pull-requests: read
-    outputs:
-      python: ${{ steps.filter.outputs.python }}
-    steps:
-      - uses: dorny/paths-filter@v3
-        id: filter
-        with:
-          filters: |
-            python:
-            - '**/*.py'
-            - 'pyproject.toml'
+  # changes:
+  #   runs-on: ubuntu-latest
+  #   permissions:
+  #     pull-requests: read
+  #   outputs:
+  #     python: ${{ steps.filter.outputs.python }}
+  #   steps:
+  #     - uses: dorny/paths-filter@v3
+  #       id: filter
+  #       with:
+  #         filters: |
+  #           python:
+  #           - '**/*.py'
+  #           - 'pyproject.toml'
 
   test-python:
     runs-on: ${{ matrix.os }}
@@ -34,25 +34,27 @@ jobs:
       fail-fast: false
       matrix:
         py: ['3.11', '3.12']
-        os: [ubuntu-latest, macos-latest, windows-latest]
+        # not today, windows-latest
+        os: [ubuntu-latest, macos-latest]
 
-    needs: changes
-    if: needs.changes.outputs.python == 'true'
+    # needs: changes
+    # if: needs.changes.outputs.python == 'true'
     steps:
     - uses: actions/checkout@v4
-    - uses: eifinger/setup-rye@v3
+    - uses: eifinger/setup-rye@v4  # replaces setup-python
       id: setup-rye
       with:
-        version: '0.34.0'
+        version: '0.39.0'
     - run: rye pin ${{ matrix.py }}
     - name: Sync
+      shell: pwsh
       run: |
         rye sync
-        if [[ $(git diff --stat requirements.lock) != '' ]]; then
-            echo 'Rye lockfile not up-to-date'
+        if (git diff --stat requirements.lock) {
+            Write-Output 'Rye lockfile not up-to-date'
             git diff requirements.lock
             exit 1
-        fi
+        }
     - run: rye fmt --check
     - run: rye lint
     - run: rye run check

pyproject.toml

@@ -25,6 +25,11 @@ build-backend = "pdm.backend"
 
 [tool.rye]
 managed = true
+
+# switch to true to generate cross-platform lockfiles when we support Windows
+# which we don't
+universal = false
+
 dev-dependencies = [
     "pyright>=1.1.377",
     "pytest>=8.3.2",

requirements-dev.lock

@@ -7,11 +7,15 @@
 #   all-features: false
 #   with-sources: false
 #   generate-hashes: false
-#   universal: false
+#   universal: true
 
 -e file:.
 click==8.1.7
     # via jsi
+colorama==0.4.6 ; sys_platform == 'win32' or platform_system == 'Windows'
+    # via click
+    # via loguru
+    # via pytest
 humanize==4.10.0
     # via jsi
 iniconfig==2.0.0
@@ -37,3 +41,5 @@ pyright==1.1.378
 pytest==8.3.2
 rich==13.8.1
     # via jsi
+win32-setctime==1.1.0 ; sys_platform == 'win32'
+    # via loguru

requirements.lock

@@ -26,7 +26,9 @@ psutil==6.0.0
     # via jsi
 pygments==2.18.0
     # via rich
-pyright==1.1.378
+pyright==1.1.382.post0
     # via jsi
 rich==13.8.1
     # via jsi
+typing-extensions==4.12.2
+    # via pyright

tests/test_process_control.py

@@ -81,7 +81,7 @@ def test_cmd_options():
         stderr="boop",
     )
     command.start()
-    stdout, stderr = command.communicate(timeout=0.1)
+    stdout, stderr = command.communicate(timeout=0.2)
 
     print(f"{stdout=}")
     print(f"{stderr=}")
@@ -165,7 +165,8 @@ def test_delayed_start_real_time():
     assert not command.started()
     assert not command.done()
 
-    time.sleep(0.2)
+    # give it some time to complete (allow some wiggle room for slow CI)
+    time.sleep(0.4)
     assert command.started()
     assert command.done()
     assert command.returncode == 0