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 dependencies yargs and yargs-parser #4543

Merged
merged 2 commits into from
Jan 18, 2021
Merged

Update dependencies yargs and yargs-parser #4543

merged 2 commits into from
Jan 18, 2021

Conversation

juergba
Copy link
Contributor

@juergba juergba commented Jan 5, 2021

Description

We have been using rather old versions of these dependencies. It's time for an upgrade.

There have been breaking changes in yargs' handling of array typed options, especially when supplied via configuration object config(). In order to avoid double entries when combining array values, I had to remove aliased options from the parsed result.

@juergba juergba added dependencies Pull requests that update a dependency file semver-patch implementation requires increase of "patch" version number; "bug fixes" labels Jan 5, 2021
@juergba juergba self-assigned this Jan 5, 2021
@coveralls
Copy link

coveralls commented Jan 5, 2021

Coverage Status

Coverage remained the same at 94.143% when pulling 99529b4 on juergba/yargs into c21a90f on master.

@juergba juergba force-pushed the juergba/yargs branch 7 times, most recently from db8425f to 8533ade Compare January 10, 2021 16:50
@juergba juergba requested a review from a team January 10, 2021 17:11
This was referenced Mar 15, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file semver-patch implementation requires increase of "patch" version number; "bug fixes"
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants