cI: Add more renovate rules based on having done a scan. - #641
cI: Add more renovate rules based on having done a scan.#641thompson-tomo wants to merge 4 commits into
Conversation
|
Thanks for working on this. I think Renovate is useful for this repo, but this change enables too much at once. Can we keep dashboard approval as the default and enable each dependency type separately after reviewing what Renovate finds? I also suggest keeping digest updates separate. Combining Docker, GitHub Actions, Bazel, and library updates in one PR could make CI failures difficult to understand and may block unrelated updates. A safer approach would be:
This will give us the benefits of Renovate without creating large mixed update PRs. |
Thats exactly what i did, renovate didn't find as much as i was hoping for but thats due to using apt packages, git repo's.
Only patches/digest are combined everything else is seperate. If a patch breaks ci, then something is wrong as that shouldn't be the case.
Have just put more behind approval.
Already the case except for patches.
Most updates are already seperate, key thing i like to avoid is having a too noisy setup especially when review bandwidth is low. Shortly after the config pr is merged the issue will show you its update plan. |
|
@lalitb the dashboard is available as #672 and even with these rules a very significant number of updates will still be behind approval (all majors) and in fact most updates will still be single updates as they are minor. Let me know if there is any further feedback after seeing the dashboard & the pool of updates. |
This analyses the renovate scan to implement rules to help mantain the dependencies going forward.
A balance between grouping and individual pr's will be raised to minimise updates blocked due to failing ci.
Each rule has a description of what it does.
This is a Continuation of #640 which enabled the scanning.