Skip to content

👷 [RUM-15055] pin GitHub Actions to full commit SHAs#60

Merged
bcaudan merged 1 commit into
mainfrom
bcaudan/fix-npm-publish
Mar 26, 2026
Merged

👷 [RUM-15055] pin GitHub Actions to full commit SHAs#60
bcaudan merged 1 commit into
mainfrom
bcaudan/fix-npm-publish

Conversation

@bcaudan

@bcaudan bcaudan commented Mar 26, 2026

Copy link
Copy Markdown
Collaborator

Motivation

The DataDog enterprise policy requires all GitHub Actions to be pinned to a full-length commit SHA. After merging #56, the tag-release workflow was failing with:

The action actions/checkout@v4 is not allowed ... all actions must also be pinned to a full-length commit SHA.

Changes

  • Pin actions/checkout@v411bd71901 (v4.2.2) in both tag-release.yml and publish.yml
  • Pin actions/setup-node@v449933ea52 (v4.4.0) in publish.yml
  • Document SHA pinning convention in docs/DEVELOPMENT.md under Dependency Management

Test instructions

Push to main and verify tag-release workflow runs without the enterprise policy error. Manually trigger publish workflow to confirm it passes the action allowlist check.

Checklist

  • Tested locally (playground)
  • Added unit tests for this change.
  • Added e2e/integration tests for this change.
  • Updated related documentation.

- pin actions/checkout to 11bd71901 (v4.2.2) in both workflows
- pin actions/setup-node to 49933ea52 (v4.4.0) in publish.yml
- document SHA pinning convention in DEVELOPMENT.md
@bcaudan bcaudan marked this pull request as ready for review March 26, 2026 08:35
@bcaudan bcaudan requested a review from a team as a code owner March 26, 2026 08:35
@bcaudan bcaudan merged commit fd4c8cb into main Mar 26, 2026
12 checks passed
@bcaudan bcaudan deleted the bcaudan/fix-npm-publish branch March 26, 2026 14:00
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