-
Notifications
You must be signed in to change notification settings - Fork 43
chore(ci): use renovate to keep deps up to date #76
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
base: main
Are you sure you want to change the base?
Conversation
4348354
to
8e0f653
Compare
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.
Thanks @nickhudkins, I like using renovate for this makes a lot of sense to me, left some minor comments. 👍
Update renovate.json Update requirements.txt
562aad1
to
3b7b732
Compare
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.
Awesome, thanks again for taking this on @nickhudkins 🚀
"dockerfile", | ||
"pip_requirements" | ||
], | ||
"dockerfile": { |
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.
hm, I don't see this option as a configuration option here: https://docs.renovatebot.com/configuration-options. Am I missing something here?
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 a bit weird, and I am not totally sure that I need it since the enabledManagers
array has it as well, but there is also this guidance about enabling/disabling managers :
https://docs.renovatebot.com/modules/manager/#enabling-and-disabling-managers
Updated renovate config and moved requirements to independent file to allow renovate to "Just Work™"