Skip to content

chore: disable npm caching in release workflow#460

Merged
aidandaly24 merged 1 commit intomainfrom
chore/disable-release-workflow-caching
Feb 26, 2026
Merged

chore: disable npm caching in release workflow#460
aidandaly24 merged 1 commit intomainfrom
chore/disable-release-workflow-caching

Conversation

@tejaskash
Copy link
Contributor

@tejaskash tejaskash commented Feb 26, 2026

Summary

  • Removes cache: 'npm' from all three jobs in the release workflow (prepare-release, test-and-build, publish-npm)
  • Ensures fresh npm ci installs during releases with no cached dependencies
  • Addresses security review feedback to eliminate cached artifact reuse in the release pipeline

Test plan

  • Trigger a dry-run of the release workflow and verify all three jobs install dependencies fresh
  • Confirm other workflows (lint, build-and-test, e2e) still use caching as before

Remove `cache: 'npm'` from all three jobs in the release workflow
(prepare-release, test-and-build, publish-npm) to ensure fresh
dependency installs during releases per security review.
@tejaskash tejaskash requested a review from a team February 26, 2026 15:50
@github-actions github-actions bot added the size/xs PR size: XS label Feb 26, 2026
Copy link
Contributor

@aidandaly24 aidandaly24 left a comment

Choose a reason for hiding this comment

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

lgtm approved

@aidandaly24 aidandaly24 merged commit ca5644f into main Feb 26, 2026
17 of 20 checks passed
@aidandaly24 aidandaly24 deleted the chore/disable-release-workflow-caching branch February 26, 2026 15:53
@github-actions
Copy link
Contributor

Coverage Report

Status Category Percentage Covered / Total
🔵 Lines 44.28% 3021 / 6822
🔵 Statements 43.84% 3180 / 7252
🔵 Functions 45.93% 622 / 1354
🔵 Branches 48.74% 1968 / 4037
Generated in workflow #748 for commit 20ebb97 by the Vitest Coverage Report Action

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size/xs PR size: XS

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants