-
Notifications
You must be signed in to change notification settings - Fork 1
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
Third-party dependency upgrades are not delayed by 7 days #2
Comments
@onebytegone The documentation suggests adding the I had a slight suspicion that Renovate bot could be unsure whether to use I'm not sure how to test though, but I'll open the PR for it. |
@MatheusBaldi It seems the changes in #3 has made renovatebot replace the by-package PRs (e.g. silvermine/toolbox#96) with one "update all the packages" PR (e.g. silvermine/toolbox#99). Able to investigate why? It would be ideal to keep the by-package PRs. |
@onebytegone It is possibly a behavior coming from the config:recommended preset that's being extended renovatebot/renovate#24071 (reply in thread). I've read some of their docs and this solutions seems worth trying. Here are some of the resources: How presets work: https://docs.renovatebot.com/config-presets/ I'll open a PR with this solution. If it works, but only a few pull requests are being created when there are many more packages to be updated, there is a follow up solution in the discussion referenced above that can be added in a future PR renovatebot/renovate#24071 (reply in thread). |
@MatheusBaldi The default behavior of grouping monorepos is what we want (e.g. all |
@onebytegone it makes sense, I'll change the PR. |
In the config we specify that most packages shouldn't be upgraded for 7 days. However, in cases like onebytegone/cloud-utils#39, the PR for the dependency upgrade was submitted the same day as the new version.
Desired behavior
@silvermine/*
packages can be created immediatelyThe text was updated successfully, but these errors were encountered: