Renovate ignores Git fixup commits #33658
Replies: 2 comments 1 reply
-
Renovate only checks the commit author and it's probably not easily doable / desired to check the committer renovate/lib/util/git/index.ts Line 674 in 1d64a10 |
Beta Was this translation helpful? Give feedback.
-
Hi there, Get your discussion fixed faster by creating a minimal reproduction. This means a repository dedicated to reproducing this issue with the minimal dependencies and config possible. Before we start working on your issue we need to know exactly what's causing the current behavior. A minimal reproduction helps us with this. Discussions without reproductions are less likely to be converted to Issues. Please follow these steps:
If you need help with running renovate on your minimal reproduction repository, please refer to our Running Renovate guide. The Renovate team |
Beta Was this translation helpful? Give feedback.
-
How are you running Renovate?
Self-hosted Renovate
If you're self-hosting Renovate, tell us which platform (GitHub, GitLab, etc) and which version of Renovate.
GitHub, Renovate 39
Please tell us more about your question or problem
Sometimes, PRs created by Renovate need to be manually extended. Normally, Renovate detects additional commits created by other users and does not rebase the PR.
However, we sometimes use the Git fixup method (see e.g. https://mikulskibartosz.name/git-fixup-explained) to extend commits created by Renovate. Using this method results in a single commit in the PR, which is authored by renovate as well as the user that extended it.
In this case, Renovate does not detect that the PR was manually modified. Therefore, when Renovate runs the next time, it rebases the PR and throws away the extended commit. We can work around this problem by adding the
stop-updating
label to the PR. However, people have to think about this.Is it possible to change the way how Renovate decides if a PR needs rebasing so that it is only rebased if Renovate is the sole author of the commits in the PR?
Logs (if relevant)
Logs
Beta Was this translation helpful? Give feedback.
All reactions