Skip to content

ci: pin test-summary/action to v2.4 - #726

Open
HGyllensvard wants to merge 1 commit into
masterfrom
fix/test-summary-action-v2.4
Open

ci: pin test-summary/action to v2.4#726
HGyllensvard wants to merge 1 commit into
masterfrom
fix/test-summary-action-v2.4

Conversation

@HGyllensvard

Copy link
Copy Markdown
Contributor

v2 tag was re-pointed to an unbuilt commit missing index.js. v2.4 is the latest tag with the compiled artifact.

See test-summary/action#75

v2 tag was re-pointed to an unbuilt commit missing index.js.
v2.4 is the latest tag with the compiled artifact.

See test-summary/action#75

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@HGyllensvard
HGyllensvard requested a review from a team as a code owner May 8, 2026 00:08

@NikoRoberts NikoRoberts left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This tightens test-summary/action from the v2 major tag to v2.4, which is a small improvement.

One thing to flag: test-summary/action is a third-party action, and both v2 and v2.4 are mutable tags that can be re-pointed by the maintainer. For third-party actions the safer posture is pinning to a full commit SHA. This is not blocking here since the action only reads test report XML and has no access to deploy secrets, but it is worth considering if we standardise action pinning across repos.

Otherwise this is fine to merge once checks pass.

Automated review, queued by Niko's agents

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.

2 participants