Skip to content

Fix/download artifact test unit.yml#6572

Merged
jyotshnayaparla-00 merged 1 commit into
mainfrom
fix/download-artifact-test-unit
Jul 2, 2026
Merged

Fix/download artifact test unit.yml#6572
jyotshnayaparla-00 merged 1 commit into
mainfrom
fix/download-artifact-test-unit

Conversation

@jyotshnayaparla-00

Copy link
Copy Markdown
Contributor

Switches test-unit.yml from actions/download-artifact to dawidd6/action-download-artifact, same as the test-adapters.yml

@jyotshnayaparla-00 jyotshnayaparla-00 self-assigned this Jul 2, 2026
@jyotshnayaparla-00
jyotshnayaparla-00 force-pushed the fix/download-artifact-test-unit branch from c5adad2 to e3f10e7 Compare July 2, 2026 00:20
@jyotshnayaparla-00
jyotshnayaparla-00 force-pushed the fix/download-artifact-test-unit branch from e3f10e7 to 692f15c Compare July 2, 2026 00:29
@jyotshnayaparla-00
jyotshnayaparla-00 added this pull request to the merge queue Jul 2, 2026

@mythical-fred mythical-fred left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Mirrors what test-adapters.yml already does, action is pinned by commit SHA, and passing GITHUB_TOKEN explicitly is required by dawidd6/action-download-artifact.

Minor follow-up (not blocking): test-adapters.yml also exposes a workflow_dispatch input run_id and threads it via ${{ github.event.inputs.run_id || github.run_id }}, which is handy when you want to re-run just the unit tests against a previous build's artifacts. Worth adding here for symmetry, but a separate PR.

Merged via the queue into main with commit ee0617c Jul 2, 2026
1 check passed
@jyotshnayaparla-00
jyotshnayaparla-00 deleted the fix/download-artifact-test-unit branch July 2, 2026 02:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants