-
Notifications
You must be signed in to change notification settings - Fork 34
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
fix(deps): update dependency yargs to v13 #260
Conversation
@@ -17,7 +17,7 @@ | |||
"@google-cloud/logging-bunyan": "^0.9.5", | |||
"bunyan": "^1.8.12", | |||
"express": "^4.16.3", | |||
"yargs": "^12.0.0" | |||
"yargs": "^13.0.0" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Does it make sense for this to be in devDependencies instead? (/tangential)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It's all fake money, since these modules never get published. That having been said - yargs is required to run the sample itself, so I'd say regular deps sounds right.
908680e
to
b75536a
Compare
Codecov Report
@@ Coverage Diff @@
## master #260 +/- ##
=======================================
Coverage 91.78% 91.78%
=======================================
Files 2 2
Lines 73 73
Branches 6 6
=======================================
Hits 67 67
Misses 6 6 Continue to review full report at Codecov.
|
b75536a
to
b2af3ed
Compare
This PR contains the following updates:
^12.0.0
->^13.0.0
Release Notes
yargs/yargs
v13.1.0
Compare Source
Features
Renovate configuration
📅 Schedule: "after 9am and before 3pm" (UTC).
🚦 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.
This PR has been generated by Renovate Bot. View repository job log here.