Skip to content

maintenance: add auto-updating of npm packages#1626

Merged
callmehiphop merged 5 commits intogoogleapis:masterfrom
stephenplusplus:spp--update-all
Sep 26, 2016
Merged

maintenance: add auto-updating of npm packages#1626
callmehiphop merged 5 commits intogoogleapis:masterfrom
stephenplusplus:spp--update-all

Conversation

@stephenplusplus
Copy link
Contributor

@stephenplusplus stephenplusplus commented Sep 26, 2016 •
edited
Loading

This script will update our sub-modules to the latest compatible dependencies when run:

$ npm run update-deps

Also included in the PR are the modified package.json's after running.

@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Sep 26, 2016
stephenplusplus
stephenplusplus commented Sep 26, 2016
View reviewed changes
packages/dns/package.json Outdated
"@google-cloud/common": "^0.5.0",
"arrify": "^1.0.0",
"dns-zonefile": "0.1.18",
"dns-zonefile": "^0.1.18",

This comment was marked as spam.

Sign in to view

This comment was marked as spam.

Sign in to view

This comment was marked as spam.

Sign in to view

@coveralls
Copy link

coveralls commented Sep 26, 2016

Coverage Status

Changes Unknown when pulling 14a34bd on stephenplusplus:spp--update-all into * on GoogleCloudPlatform:master*.

callmehiphop
callmehiphop reviewed Sep 26, 2016
View reviewed changes
scripts/update-deps.js
'use strict';

var async = require('async');
var david = require('david');

This comment was marked as spam.

Sign in to view

This comment was marked as spam.

Sign in to view

@callmehiphop
Copy link
Contributor

callmehiphop commented Sep 26, 2016

So what's the long term plan for this? Run it once in a while to see if we have updates..? I think it would be great it we could tie this into CI somewhere and have it just run weekly or so..?

@stephenplusplus
Copy link
Contributor Author

stephenplusplus commented Sep 26, 2016

That's interesting. My plan was to simply run it manually before publishing.

@callmehiphop callmehiphop merged commit 25eab56 into googleapis:master Sep 26, 2016
sofisl pushed a commit that referenced this pull request Nov 17, 2022
@stephenplusplus @callmehiphop
maintenance: add auto-updating of npm packages (#1626)
9d221a5
@sofisl sofisl mentioned this pull request Nov 17, 2022
Merged
4 tasks
@sofisl sofisl mentioned this pull request Jan 17, 2023
Merged
4 tasks
sofisl pushed a commit that referenced this pull request Jan 17, 2023
@stephenplusplus @callmehiphop
maintenance: add auto-updating of npm packages (#1626)
64b5baa
sofisl pushed a commit that referenced this pull request Jan 24, 2023
@stephenplusplus @callmehiphop
maintenance: add auto-updating of npm packages (#1626)
ef3b48b
sofisl pushed a commit that referenced this pull request Jan 25, 2023
@stephenplusplus @callmehiphop
maintenance: add auto-updating of npm packages (#1626)
f8c5ec0
miguelvelezsa pushed a commit that referenced this pull request Jul 23, 2025
@ddelgrosso1
fix(deps): remove rimraf in favor of native node rm function (#1626)
b4c2b45 
* fix(deps): remove rimraf in favor of native node rm function

* use fsp.rm instead of rmSync

* pin additional deps

* move pinned deps to devDeps
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer

@callmehiphop callmehiphop callmehiphop left review comments

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

cla: yes This human has signed the Contributor License Agreement.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@stephenplusplus @coveralls @callmehiphop @googlebot