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 2 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. |
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.