Rename build.yml, as it doesn't build anything. Only test on Windows and MacOS in parent repo, due to limited Github hosted runner availability for forks.

JamesParrott · JamesParrott · commit 8c9bd567ebbe · 2024-09-17T16:42:07.000+01:00
diff --git a/.github/workflows/deploy.yml b/.github/workflows/deploy.yml
@@ -23,13 +23,24 @@ jobs:
       uses: actions/setup-python@v2
       with:
         python-version: '3.x'
+
+    # In general tests should be run after building a distribution, to test that distribution.
+    # However as long as PyShp is a pure Python library, with pure Python deps (or no deps)
+    # then it's not crucial.
+
+    # Prevent deployment of releases that fail any hooks (e.g. linting) or that fail any tests.
+    - name: Run tests and hooks
+      uses: ./.github/workflows/run_tests_and_hooks.yml
+
     - name: Install dependencies
       run: |
         python -m pip install --upgrade pip
         pip install build
     - name: Build package
       run: python -m build
+
     - name: Publish package
+      if: github.repository == 'GeospatialPython/pyshp'
       uses: pypa/gh-action-pypi-publish@27b31702a0e7fc50959f5ad993c78deac1bdfc29
       with:
         user: __token__
diff --git a/.github/workflows/run_tests_and_hooks.yml b/.github/workflows/run_tests_and_hooks.yml
@@ -1,11 +1,12 @@
 # This workflow will run the pre-commit hooks (including linters), and the tests with a variety of Python versions
 
-name: build
+name: Run pre-commit hooks and tests
 
 on:
   push:
   pull_request:
     branches: [ master ]
+  workflow_call:
   workflow_dispatch:
 
 jobs:
@@ -45,6 +46,7 @@ jobs:
 
 
   test_on_MacOS_and_Windows:
+    if: github.repository == 'GeospatialPython/pyshp'
     strategy:
       fail-fast: false
       matrix:
