Skip to content

maintenance: add auto-updating of npm packages #1626

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.

Sign up for GitHub

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

Jump to bottom
Merged
callmehiphop merged 5 commits into from
Sep 26, 2016
Merged

maintenance: add auto-updating of npm packages #1626

callmehiphop merged 5 commits into from
Sep 26, 2016

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