Skip to content

[ci] Gate worker CLI versions on npm availability - #4313

Merged
sjchmiela merged 2 commits into
mainfrom
stanley/guard-eas-cli-release-availability
Sep 2, 2026
Merged

[ci] Gate worker CLI versions on npm availability#4313
sjchmiela merged 2 commits into
mainfrom
stanley/guard-eas-cli-release-availability

Conversation

@sjchmiela

@sjchmiela sjchmiela commented Aug 31, 2026

Copy link
Copy Markdown
Contributor

We currently:

  • publish to NPM without awaiting the release being ready and public
  • immediately change cli-versions.json
  • this makes builds immediately try to use a new eas-cli version despite it NOT being public yet, which causes problems, see Datadog.

This pull request makes it so we:

  • publish to NPM, make a GitHub release
  • update changelog and push to main
  • await NPM readiness for 10 minutes (last release took 6)
  • only once it's ready, update cli-versions.json.

@codecov

codecov Bot commented Aug 31, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 65.46%. Comparing base (eebe02e) to head (b37f589).

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #4313   +/-   ##
=======================================
  Coverage   65.46%   65.46%           
=======================================
  Files        1060     1060           
  Lines       49291    49291           
  Branches    10427    10427           
=======================================
  Hits        32264    32264           
  Misses      16911    16911           
  Partials      116      116           

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@sjchmiela sjchmiela added the no changelog PR that doesn't require a changelog entry label Aug 31, 2026
@sjchmiela
sjchmiela enabled auto-merge (squash) September 1, 2026 07:28
@sjchmiela
sjchmiela force-pushed the stanley/guard-eas-cli-release-availability branch from 72e1f72 to b37f589 Compare September 1, 2026 07:28
@sjchmiela
sjchmiela force-pushed the stanley/guard-eas-cli-release-availability branch from b37f589 to 69455ec Compare September 2, 2026 08:59
@github-actions

github-actions Bot commented Sep 2, 2026

Copy link
Copy Markdown

⏩ The changelog entry check has been skipped since the "no changelog" label is present.

@sjchmiela
sjchmiela merged commit 7727b9c into main Sep 2, 2026
6 of 7 checks passed
@sjchmiela
sjchmiela deleted the stanley/guard-eas-cli-release-availability branch September 2, 2026 09:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

no changelog PR that doesn't require a changelog entry

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants