Skip to content

fix: use a version range for cli-core#854

Merged
binoy14 merged 1 commit intomainfrom
03-31-fix_use_a_version_range_for_cli-core
Mar 31, 2026
Merged

fix: use a version range for cli-core#854
binoy14 merged 1 commit intomainfrom
03-31-fix_use_a_version_range_for_cli-core

Conversation

@binoy14
Copy link
Copy Markdown
Contributor

@binoy14 binoy14 commented Mar 31, 2026

Uses a caret range for cli-core instead of a pin version as versions are not always together

Copy link
Copy Markdown
Contributor Author

binoy14 commented Mar 31, 2026

This stack of pull requests is managed by Graphite. Learn more about stacking.

@claude
Copy link
Copy Markdown

claude bot commented Mar 31, 2026

Claude finished @binoy14's task in 36s —— View job


Review

No issues found. The change from workspace:* to workspace:^ for @sanity/cli-core is correct — workspace:^ allows compatible minor/patch updates within the workspace, which is the standard pnpm convention for internal packages that don't need to be pinned to an exact version.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Mar 31, 2026

📦 Bundle Stats — @sanity/cli

Compared against main (d7534ed8) · v6.3.0 (npm)

@sanity/cli

Metric Value vs main (d7534ed) vs v6.3.0
Internal (raw) 2.1 KB - -
Internal (gzip) 799 B - -
Bundled (raw) 11.06 MB - +9.12 MB, +470.6%
Bundled (gzip) 2.07 MB - +1.60 MB, +335.7%
Import time 845ms +4ms, +0.5% +38ms, +4.7%

bin:sanity

Metric Value vs main (d7534ed) vs v6.3.0
Internal (raw) 975 B - -
Internal (gzip) 460 B - -
Bundled (raw) 9.83 MB - +9.12 MB, +1286.7%
Bundled (gzip) 1.77 MB - +1.60 MB, +940.0%
Import time NaNs - NaNs, NaN%

🗺️ View treemap · Artifacts

Details
  • Import time regressions over 10% are flagged with ⚠️
  • Sizes shown as raw / gzip 🗜️. Internal bytes = own code only. Total bytes = with all dependencies. Import time = Node.js cold-start median.

📦 Bundle Stats — @sanity/cli-core

Compared against main (d7534ed8) · v1.3.0 (npm)

Metric Value vs main (d7534ed) vs v1.3.0
Internal (raw) 92.2 KB - -
Internal (gzip) 21.6 KB - -
Bundled (raw) 21.64 MB - +9.13 MB, +72.9%
Bundled (gzip) 3.43 MB - +1.59 MB, +86.9%
Import time 791ms -6ms, -0.7% +43ms, +5.8%

🗺️ View treemap · Artifacts

Details
  • Import time regressions over 10% are flagged with ⚠️
  • Sizes shown as raw / gzip 🗜️. Internal bytes = own code only. Total bytes = with all dependencies. Import time = Node.js cold-start median.

@binoy14 binoy14 marked this pull request as ready for review March 31, 2026 16:34
@binoy14 binoy14 requested a review from a team as a code owner March 31, 2026 16:34
@binoy14 binoy14 requested review from bjoerge and gu-stav and removed request for a team March 31, 2026 16:34
@binoy14 binoy14 enabled auto-merge (squash) March 31, 2026 16:41
@github-actions
Copy link
Copy Markdown
Contributor

Coverage Delta

No covered files changed in this PR.

Overall Coverage

Metric Coverage
Statements 83.1% (±0%)
Branches 72.9% (±0%)
Functions 83.3% (±0%)
Lines 83.5% (±0%)

@binoy14 binoy14 merged commit 06f62f8 into main Mar 31, 2026
44 checks passed
@binoy14 binoy14 deleted the 03-31-fix_use_a_version_range_for_cli-core branch March 31, 2026 16:41
@squiggler-app squiggler-app bot mentioned this pull request Mar 31, 2026
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