Skip to content

Comments

[release] Work around the removal of yarn --silent#3424

Merged
douglowder merged 1 commit intomainfrom
doug/fix-lerna-release
Feb 24, 2026
Merged

[release] Work around the removal of yarn --silent#3424
douglowder merged 1 commit intomainfrom
doug/fix-lerna-release

Conversation

@douglowder
Copy link
Contributor

@douglowder douglowder commented Feb 24, 2026

Why

When yarn moved to version 4, they removed the --silent flag. Because of this change, any use of yarn to call scripts and get output will also get yarn debugging garbage, and there is no way to suppress it.

The release scripts are mostly ok with this, except for calling the next-version script. The output of this script has to be clean for it to be used as input for lerna version.

How

Replaced the yarn call with a direct invocation of the node Typescript source, bypassing yarn.

Test Plan

Since the trigger-release.yml workflow supports a dry run mode, it's safe to modify it to check out a test branch and run the workflow many times to debug the issue.

Once this change is merged, a dry run invocation of the workflow should succeed even if no version string is passed in.

Copy link
Contributor Author

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

@douglowder douglowder marked this pull request as ready for review February 24, 2026 03:35
@douglowder douglowder added the no changelog PR that doesn't require a changelog entry label Feb 24, 2026 — with Graphite App
@github-actions
Copy link

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

@codecov
Copy link

codecov bot commented Feb 24, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 52.32%. Comparing base (3e13df4) to head (fc905a2).
⚠️ Report is 2 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #3424      +/-   ##
==========================================
- Coverage   52.60%   52.32%   -0.27%     
==========================================
  Files         806      807       +1     
  Lines       33604    33868     +264     
  Branches     6985     7033      +48     
==========================================
+ Hits        17673    17719      +46     
- Misses      14548    14750     +202     
- Partials     1383     1399      +16     

☔ View full report in Codecov by Sentry.
📢 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.

Copy link
Contributor Author

douglowder commented Feb 24, 2026

Merge activity

  • Feb 24, 11:37 AM UTC: A user started a stack merge that includes this pull request via Graphite.
  • Feb 24, 11:37 AM UTC: @douglowder merged this pull request with Graphite.

@douglowder douglowder merged commit 0c32aa3 into main Feb 24, 2026
13 of 14 checks passed
@douglowder douglowder deleted the doug/fix-lerna-release branch February 24, 2026 11:37
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