Skip to content

Use npm ci --ignore-scripts in update_schemastore.py#16915

Merged
woodruffw merged 2 commits intomainfrom
ww/npm
Dec 1, 2025
Merged

Use npm ci --ignore-scripts in update_schemastore.py#16915
woodruffw merged 2 commits intomainfrom
ww/npm

Conversation

@woodruffw
Copy link
Member

Summary

This ensures that we fully honor SchemaStore's lockfile when installing, plus that we don't run any build-time scripts (which shouldn't be needed here).

Test Plan

I ran uv run update_schemastore.py locally and confirmed that it created a branch as expected.

Signed-off-by: William Woodruff <william@astral.sh>
@woodruffw woodruffw self-assigned this Dec 1, 2025
@woodruffw woodruffw temporarily deployed to uv-test-registries December 1, 2025 21:29 — with GitHub Actions Inactive
@woodruffw woodruffw added the internal A refactor or improvement that is not user-facing label Dec 1, 2025
Signed-off-by: William Woodruff <william@astral.sh>
@woodruffw woodruffw temporarily deployed to uv-test-registries December 1, 2025 23:26 — with GitHub Actions Inactive
@woodruffw woodruffw enabled auto-merge (squash) December 1, 2025 23:35
@woodruffw woodruffw merged commit 3347e19 into main Dec 1, 2025
101 checks passed
@woodruffw woodruffw deleted the ww/npm branch December 1, 2025 23:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

internal A refactor or improvement that is not user-facing

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants