Skip to content
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

Streamline renovate behavior and config #314

Merged
merged 5 commits into from
Dec 17, 2024
Merged

Streamline renovate behavior and config #314

merged 5 commits into from
Dec 17, 2024

Conversation

martis42
Copy link
Owner

For details see commits

Less noisy style for enabling basic features and getting all sane defaults.
We don't set the `deps` label when we manually update dependencies. The
renovate bot should use a label we can use to fastly find its PRs. But,
the name should make clear its about the renovate bot, not generally
"all dependency thingies".
We duplicate `com_google_protobuf` with `protobuf` as it seems renovate
is not properly ignoring based on `com_google_protobuf` alone.
While it is annoying with our CI setup (only PRs rebased to HEAD can merge)
to have several PRs for trivial dev dependency updates, this creates a
cleaner git history.
Unfortunately, renovate is not able to combine multiple commits into a
single PR to achieve efficient CI usage and minimal noise with a clean
git history.
We should only update our dev dependencies automatically, not our
production dependencies. Our production dependencies should be close to the
minimum version required for DWYU to work.
@martis42 martis42 merged commit cf7c2a3 into main Dec 17, 2024
11 checks passed
@martis42 martis42 deleted the streamline_renovate branch December 17, 2024 09:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant