Merge pull request #1728 from phargogh/task/1718-invest-3.14.3-release

Fixing an artifact name in release automation
natcap · Jan 10, 2025 · 4af6cd2 · 4af6cd2
2 parents 75c47b6 + e5a3065
commit 4af6cd2
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 2 deletions.
diff --git a/.github/workflows/build-and-test.yml b/.github/workflows/build-and-test.yml
@@ -198,6 +198,9 @@ jobs:
 
       - uses: actions/upload-artifact@v4
         with:
+          # NOTE: if you change this name, make sure the source distribution
+          # pattern defined in .github/workflows/release-part-2.yml will still
+          # grab it!
           name: Source distribution ${{ matrix.os }} ${{ matrix.python-version }}
           path: dist
 

diff --git a/.github/workflows/release-part-2.yml b/.github/workflows/release-part-2.yml
@@ -75,12 +75,18 @@ jobs:
           for artifact in Workbench-Windows-binary \
                           Workbench-macOS-binary \
                           InVEST-sample-data \
-                          InVEST-user-guide \
-                          "Source distribution"
+                          InVEST-user-guide
           do
             gh run download $RUN_ID --dir artifacts --name "$artifact"
           done
 
+          # We build one sdist per combination of OS and python version, so just
+          # download and unzip all of them into an sdists directory so we can
+          # just grab the first one.  This approach is more flexible to changes
+          # in OS and python versions than just statically defining the artifact name.
+          gh run download $RUN_ID --dir sdists --pattern "Source*"
+          cp "$(find sdists -name '*.tar.gz' -print -quit)" artifacts/
+
       - name: Create Github release
         run: |
           # Copy the history notes for this version into the release message