Is it possible to adjust the PR description generated by renovate to better
align with what you're looking for? The docs
<https://docs.renovatebot.com/configuration-templates/#pr-body> seem to
suggest that it can be done with e.g. {{{commitList}}}

--EM

On Wed, Jul 9, 2025 at 9:08 AM Robert Stupp <sn...@snazy.de> wrote:

> My biggest concern is the Renovate PRs. The commit message of those is
> suitable for being merged "as is" - but the PR description is not.
> Given the amount of Renovate-PRs it's a lot of cognitive work to
> adjust those - or the commit log would look really really awful with a
> ton of not useful information and links - leaking into (automatically
> generated) changelogs for releases.
>
> All "human originated" PRs need manual attention anyway.
>
>
> On Wed, Jul 9, 2025 at 5:54 PM Eric Maynard <eric.w.mayn...@gmail.com>
> wrote:
> >
> > Hi Robert,
> >
> > > Having the list of commits in the merge commit message proposed by
> GitHub
> > helps there.
> >
> > Can you elaborate on this? How exactly does it help? You can always view
> > the commits on the original PR.
> >
> > Beyond that, I actually don't see these commit messages being added
> > currently when I look at recently merged PRs like 1
> > <
> https://github.com/apache/polaris/commit/a0e31b0584437602f30a97ed5fc110c50802d9c1
> >
> >  or 2
> > <
> https://github.com/apache/polaris/commit/a0e31b0584437602f30a97ed5fc110c50802d9c1
> >.
> > They are indeed there by default right now, as you can see in this recent
> > commit
> > <
> https://github.com/apache/polaris/commit/145ad3e2715ccb4630e6537de3c077561c86a129
> >
> > which
> > includes each commit's message. But my reading of your linked comments
> > suggests that committers are meant to *change* the default commit
> message.
> >
> > Ajantha's proposed change would make the default commit message more
> > closely resemble what you suggest. In fact, for those two linked PRs, it
> > looks like the default commit message would have been exactly the same as
> > what ultimately got merged.
> >
> > --EM
> >
> > On Wed, Jul 9, 2025 at 3:09 AM Jean-Baptiste Onofré <j...@nanthrax.net>
> wrote:
> >
> > > Hi
> > >
> > > From technical standpoint, the commit template can be updated via
> asf.yaml.
> > > It’s a change we did on the parser a few months ago.
> > >
> > > Regards
> > > JB
> > >
> > > Le mer. 9 juil. 2025 à 04:30, Ajantha Bhat <ajanthab...@gmail.com> a
> > > écrit :
> > >
> > > > Hi all,
> > > > While adding the recommendation to working with the PR on site docs,
> > > > we have opened an issue
> https://github.com/apache/polaris/issues/1656 to
> > > > update the settings.
> > > >
> > > > I have discussed with ASF Infra and they mentioned, we can update by
> > > > ourselves.
> > > > Hence, I opened a PR for the same:
> > > > https://github.com/apache/polaris/pull/2016
> > > >
> > > > But it looks like we don't have consensus on this change (based on
> the
> > > > comment on the PR). Hence this discussion.
> > > >
> > > > I personally think it is good to have these settings as default as
> the
> > > > committer work will be reduced if the PR title and description is
> proper.
> > > > I don't see the drawbacks of this approach yet. Please feel free to
> > > mention
> > > > your opinion on this.
> > > >
> > > > - Ajantha
> > > >
> > >
>

Reply via email to