-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
chore: disable GreenKeeper #3014
Conversation
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.
👍 Do we still need "update-greenkeeper-json": "node bin/update-greenkeeper-json -f",
in package.json?
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.
+1 to @jannyHou's comment and should https://github.com/strongloop/loopback-next/blob/master/package.json#L80-L99 also be removed?
@jannyHou @nabdelgadir good catch! I search for other possible occurences of the term "greenkeeper" and founds few more places to change - see 63773cb LGTY now? |
Done. |
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.
cool LGTM
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.
👍
Signed-off-by: Miroslav Bajtoš <[email protected]>
63773cb
to
fa264cc
Compare
GreenKeeper does not support lerna monorepos using package-locks, see greenkeeperio/greenkeeper#1080. Let's migrate to RenovateBot, which we are already successfully using in loopback4-example-shopping.
This pull request is removing GreenKeeper configuration, scripts and documentation.
Before we land this pull request, we should probably remove GreenKeeper from GitHub applications, e.g. here: https://github.com/organizations/strongloop/settings/installations/633361