FIX: Use $GITHUB_ACTION_PATH to locate pixi.lock for downstream users

[MridulS]

I'm not sure, but I think with the change to composite action we need to also be specific about this repo's filepaths by using GITHUB_ACTION_PATH . I tried to use 0.6.0 downstream and I got an error `Failed to generate cache key: Error: ENOENT: no such file or directory, open 'pixi.lock'

Squash and merge commit message

* As the GitHub Action is being used in other workflows, it does not know the
  local filepaths and so to have prefix-dev/setup-pixi find the lock file and
  'pixi run' know where `upload_wheels.sh` is, the environmental variable
  'GITHUB_ACTION_PATH' is used through the ${{ github.action_path }} context.
   - c.f. https://docs.github.com/en/actions/writing-workflows/choosing-what-your-workflow-does/store-information-in-variables#default-environment-variables

[matthewfeickert]

Can you link to the failure downstream? If it is complaining about the lock file that seems different.

[MridulS]

https://github.com/scipp/scipp/actions/runs/11108007695/job/30860791220

[matthewfeickert]

@MridulS To see if this actually works could you (momentarily and then revert it) switch in the CI the local action path

upload-nightly-action/.github/workflows/ci.yml

Line 51 in e5087b7

uses: ./

to use your fork's MridulS-patch-1 branch?

edit: Ah, sorry I see that this is a branch on the main repo. I just misread your branch name. Let me try this quickly using my fork and your changes on a different branch.

[matthewfeickert]

@MridulS Okay, yes that works. Thank you!

What I did to test was to push your MridulS-patch-1 branch to my fork

$ git branch --all | grep patch
  MridulS-patch-1
* debug-ci-with-mriduls-patch
  matthewfeickert/MridulS-patch-1
  remotes/matthewfeickert/MridulS-patch-1
  remotes/origin/MridulS-patch-1
  remotes/origin/debug-ci-with-mriduls-patch

and then run the CI manually on another branch using my fork as the target

$ git branch
  MridulS-patch-1
* debug-ci-with-mriduls-patch
  main
  matthewfeickert/MridulS-patch-1
$ git diff origin/main 
diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
index cdd4d13..19d0427 100644
--- a/.github/workflows/ci.yml
+++ b/.github/workflows/ci.yml
@@ -48,13 +48,13 @@ jobs:
         run: python -m zipfile --list dist/test_package-*.whl
 
       - name: Test upload
-        uses: ./
+        uses: matthewfeickert/upload-nightly-action@MridulS-patch-1
         with:
           artifacts_path: dist
           anaconda_nightly_upload_token: ${{ secrets.UPLOAD_TOKEN }}
 
       - name: Test upload that forces removal first
-        uses: ./
+        uses: matthewfeickert/upload-nightly-action@MridulS-patch-1
         with:
           artifacts_path: dist
           anaconda_nightly_upload_token: ${{ secrets.UPLOAD_TOKEN }}
@@ -67,7 +67,7 @@ jobs:
           python -m build --outdir ./dist tests/test_package
 
       - name: Test upload with non-main label
-        uses: ./
+        uses: matthewfeickert/upload-nightly-action@MridulS-patch-1
         with:
           artifacts_path: dist
           anaconda_nightly_upload_token: ${{ secrets.UPLOAD_TOKEN }}
@@ -81,7 +81,7 @@ jobs:
           python -m build --outdir ./dist tests/test_package
 
       - name: Test upload with multiple labels
-        uses: ./
+        uses: matthewfeickert/upload-nightly-action@MridulS-patch-1
         with:
           artifacts_path: dist
           anaconda_nightly_upload_token: ${{ secrets.UPLOAD_TOKEN }}

That works. 👍

We need better testing though apparently.

I'm going to write some longer squash and merge commit message and then I'll merge this and get a patch release out.

[MridulS]

Thanks @matthewfeickert!

We need better testing though apparently.

Testing actions inside actions are tricky anyway.

I think it's still not done yet! We need to add --manifest-path to pixi run too! (finding new edges for pixi :) )