Skip to content

chore: pin GitHub Actions to commit SHAs#56

Merged
lvandenboom merged 1 commit into
mainfrom
chore/pin-actions-sha
Jun 15, 2026
Merged

chore: pin GitHub Actions to commit SHAs#56
lvandenboom merged 1 commit into
mainfrom
chore/pin-actions-sha

Conversation

@lvandenboom

@lvandenboom lvandenboom commented Jun 11, 2026

Copy link
Copy Markdown
Contributor

Summary of changes

  • Pins GitHub Actions and reusable workflow references to full-length commit SHAs, per GitHub's secure-use guidance for third-party actions. SHA-pinned refs carry a trailing comment with the original tag/branch for readability.
  • Also retargets the publish job from the shared publish.yml to release-pipeline.yml (the documented replacement) so manual npm publishes pass the version-type input correctly again. The previous publish.yml was repurposed to a publish-only workflow that no longer accepts version-type, which broke release runs.

Checklist

  • Added a changelog entry
  • Relevant test coverage
  • Tested and confirmed flows affected by this change are functioning as expected

Authors

List GitHub usernames for everyone who contributed to this pull request.

Reviewers

@braintree/team-sdk-js

@lvandenboom lvandenboom requested a review from a team as a code owner June 11, 2026 15:33
@github-actions

github-actions Bot commented Jun 11, 2026

Copy link
Copy Markdown

Dependency Review

The following issues were found:
  • ✅ 0 vulnerable package(s)
See the Details below.

Scanned Files

  • .github/workflows/security.yml

@lvandenboom lvandenboom force-pushed the chore/pin-actions-sha branch from f539e4d to b733c70 Compare June 12, 2026 15:06

@GoogilyBoogily GoogilyBoogily 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.

👍 🐸

@lvandenboom lvandenboom merged commit 7ff429d into main Jun 15, 2026
7 checks passed
@lvandenboom lvandenboom deleted the chore/pin-actions-sha branch June 15, 2026 16:38
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