Skip to content

Add toolcache.ts script for bundle(-from)-toolcache checks - #4255

Draft
mbg wants to merge 3 commits into
mainfrom
mbg/npm-versions
Draft

mbg wants to merge 3 commits into
mainfrom
mbg/npm-versions

Conversation

@mbg

@mbg mbg commented Oct 20, 2026

Copy link
Copy Markdown
Member

This allows the script to be shared between the two workflows and ensures that we don't have a floating npm install step in these workflows.

Risk assessment

For internal use only. Please select the risk level of this change:

  • Low risk: Changes are fully under feature flags, or have been fully tested and validated in pre-production environments and are highly observable, or are documentation or test only.

Which use cases does this change impact?

Environments:

  • Testing/None - This change does not impact any CodeQL workflows in production.

How did/will you validate this change?

  • End-to-end tests - I am depending on PR checks (i.e. tests in pr-checks).

If something goes wrong after this change is released, what are the mitigation and rollback strategies?

  • Development/testing only - This change cannot cause any failures in production.

How will you know if something goes wrong after this change is released?

  • Other - Please provide details.

Are there any special considerations for merging or releasing this change?

  • No special considerations - This change can be merged at any time.

Merge / deployment checklist

  • Confirm this change is backwards compatible with existing workflows.
  • Consider adding a changelog entry for this change.
  • Confirm the readme and docs have been updated if necessary.

@github-actions github-actions Bot added the size/M Should be of average difficulty to review label Oct 20, 2026
@mbg
mbg force-pushed the mbg/npm-versions branch from 91e76b1 to dbc9fda Compare October 20, 2026 13:15
@mbg
mbg force-pushed the mbg/npm-versions branch from dbc9fda to a4f6fc0 Compare October 20, 2026 13:18

This branch has not been deployed

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

Labels

size/M Should be of average difficulty to review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant