Skip to content

Update yargs 17.7.2 → 18.0.0 (major) - #5477

Merged
canova merged 1 commit into
mainfrom
depfu/update/yarn/yargs-18.0.0
Jun 4, 2025
Merged

Update yargs 17.7.2 → 18.0.0 (major)#5477
canova merged 1 commit into
mainfrom
depfu/update/yarn/yargs-18.0.0

Conversation

@depfu

@depfudepfuBot commented Jun 3, 2025

Copy link
Copy Markdown
Contributor

Here is everything you need to know about this upgrade. Please take a good look at what changed and the test results before merging this pull request.

What changed?

✳️ yargs (17.7.2 → 18.0.0) · Repo · Changelog

Release Notes

18.0.0

18.0.0 (2025-05-26)

⚠ BREAKING CHANGES

  • command names are not derived from modules passed to command.
  • singleton usage of yargs yargs.foo, yargs().argv, has been removed.
  • minimum node.js versions now ^20.19.0 || ^22.12.0 || >=23.
  • yargs is now ESM first

Features

Bug Fixes

  • addDirectory do not support absolute command dir (#2465) (3a40a78)
  • allows ESM modules commands to be extensible using visit option (#2468) (200e1aa)
  • browser: fix shims so that yargs continues working in browser context (#2457) (4ae5f57)
  • build: address problems with typescript compilation (#2445) (8d72fb3)
  • coerce should play well with parser configuration (#2308) (8343c66)
  • deps: update dependency yargs-parser to v22 (#2470) (639130d)
  • exit after async handler done (#2313) (e326cde)
  • handle spaces in bash completion (#2452) (83b7788)
  • parser-configuration should work well with generated completion script (#2332) (888db19)
  • propagate Dictionary including undefined in value type (#2393) (2b2f7f5)
  • zsh: completion no longer requires double tab when using autoloaded (0dd8fe4)

Code Refactoring

  • command names are not derived from modules passed to command. (d90af45)
  • singleton usage of yargs yargs.foo, yargs().argv, has been removed. (d90af45)

Build System

  • minimum node.js versions now ^20.19.0 || ^22.12.0 || >=23. (d90af45)

Does any of this look wrong? Please let us know.

Commits

See the full diff on Github. The new version differs by 33 commits:


Depfu Status

Depfu will automatically keep this PR conflict-free, as long as you don't add any commits to this branch yourself. You can also trigger a rebase manually by commenting with @depfu rebase.

All Depfu comment commands
@​depfu rebase
Rebases against your default branch and redoes this update
@​depfu recreate
Recreates this PR, overwriting any edits that you've made to it
@​depfu merge
Merges this PR once your tests are passing and conflicts are resolved
@​depfu cancel merge
Cancels automatic merging of this PR
@​depfu close
Closes this PR and deletes the branch
@​depfu reopen
Restores the branch and reopens this PR (if it's closed)
@​depfu pause
Ignores all future updates for this dependency and closes this PR
@​depfu pause [minor|major]
Ignores all future minor/major updates for this dependency and closes this PR
@​depfu resume
Future versions of this dependency will create PRs again (leaves this PR as is)

@depfudepfuBot added the dependencies Pull requests that update a dependency file label Jun 3, 2025
@depfu
depfuBot requested review from canova and julienwJune 3, 2025 14:42
@codecov

codecovBot commented Jun 3, 2025

Copy link
Copy Markdown

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 86.03%. Comparing base (1a1dc60) to head (98b8344).
Report is 2 commits behind head on main.

Additional details and impacted files
@@ Coverage Diff @@## main #5477 +/- ##
=======================================
Coverage 86.03% 86.03% =======================================
Files 312 312 Lines 30364 30364 Branches 8296 8296 =======================================
Hits 26125 26125 Misses 3644 3644 Partials 595 595 

☔ 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.

@canovacanova left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. The place we use yargs is inside the server.js file. Tested that part and it work as expected.

It looks like the major version bump removes the use of singleton usage of yargs along with some smaller changes, which we don't use.

@canova
canova merged commit caafae4 into mainJun 4, 2025
@depfu
depfuBot deleted the depfu/update/yarn/yargs-18.0.0 branch June 4, 2025 11:23
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependenciesPull requests that update a dependency file

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant

@canova