Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update dependency commander to v4 #77

Merged
merged 1 commit into from
Nov 19, 2019
Merged

Update dependency commander to v4 #77

merged 1 commit into from
Nov 19, 2019

Conversation

renovate[bot]
Copy link
Contributor

@renovate renovate bot commented Nov 1, 2019

This PR contains the following updates:

Package Type Update Change
commander dependencies major ^3.0.2 -> ^4.0.0

Release Notes

tj/commander.js

v4.0.1

Compare Source

Fixed
  • display help when requested, even if there are missing required options [(#​1091)]

v4.0.0

Compare Source

Added
  • automatically wrap and indent help descriptions for options and commands ([#​1051])
  • .exitOverride() allows override of calls to process.exit for additional error handling and to keep program running ([#​1040])
  • support for declaring required options with .requiredOptions() ([#​1071])
  • GitHub Actions support ([#​1027])
  • translation links in README
Changed
  • dev: switch tests from Sinon+Should to Jest with major rewrite of tests ([#​1035])
  • call default subcommand even when there are unknown options ([#​1047])
  • Breaking Commander is only officially supported on Node 8 and above, and requires Node 6 ([#​1053])
Fixed
  • Breaking keep command object out of program.args when action handler called ([#​1048])
    • also, action handler now passed array of unknown arguments
  • complain about unknown options when program argument supplied and action handler ([#​1049])
    • this changes parameters to command:* event to include unknown arguments
  • removed deprecated customFds option from call to child_process.spawn ([#​1052])
  • rework TypeScript declarations to bring all types into imported namespace ([#​1081])
Migration Tips
Testing for no arguments

If you were previously using code like:

if (!program.args.length) ...

a partial replacement is:

if (program.rawArgs.length < 3) ...

Renovate configuration

📅 Schedule: At any time (no schedule defined).

🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.

♻️ Rebasing: Whenever PR is stale, or if you modify the PR title to begin with "rebase!".

🔕 Ignore: Close this PR and you won't be reminded about this update again.


  • If you want to rebase/retry this PR, check this box

Newsflash: Renovate has joined WhiteSource, and is now free for all use. Learn more or view updated terms and privacy policies.

@renovate renovate bot added the internal label Nov 1, 2019
@renovate renovate bot force-pushed the renovate/commander-4.x branch 7 times, most recently from 436c393 to d2f02f7 Compare November 8, 2019 20:59
@renovate renovate bot force-pushed the renovate/commander-4.x branch 15 times, most recently from 5204dbe to 87de69a Compare November 19, 2019 07:24
@tanem tanem merged commit 47827b3 into master Nov 19, 2019
@renovate renovate bot deleted the renovate/commander-4.x branch November 19, 2019 17:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants