Re: [Python-Dev] bpo-5001: More-informative multiprocessing error messages (#3079)

30.08.17 01:52, Antoine Pitrou пише:
Please, please don't forget to edit commit messages before merging. An excessively verbose commit message will be kept in the repository forever and will harm future developers that read a history.

On 30 August 2017 at 19:39, Antoine Pitrou <solipsis@pitrou.net> wrote:
Not while we're pushing the "squash & merge" button directly, as there's no opportunity for any automated hooks to run at that point :( More options open up if the actual commit is being handled by a bot, but even that would still depend on us providing an updated commit message via some mechanism. Cheers, Nick. -- Nick Coghlan | ncoghlan@gmail.com | Brisbane, Australia

On 30 August 2017 at 10:48, Nick Coghlan <ncoghlan@gmail.com> wrote:
We could have a check for PRs that contain multiple commits. The check would flag that the commit message needs manual editing before merging, and would act as a prompt for submitters who are willing to do so to squash their PRs themselves. Personally, I'd prefer that as, apart from admin activities like making sure the PR and issue number references are present and correct, I think the original submitter has a better chance of properly summarising the PR in the merged commit message anyway. Paul

On Wed, 30 Aug 2017 at 02:56 Paul Moore <p.f.moore@gmail.com> wrote:
So you would want a comment when the PR reaches "awaiting merge" with instructions requesting the author do their own squash commit to simplify the message for us? There's also https://github.com/python/bedevere/issues/14 to help post-commit remind core devs when they forgot to do something. While it would be too late to fix a problem, hopefully regular reminders would still be helpful.

On 31 August 2017 at 20:27, Brett Cannon <brett@python.org> wrote:
That would work. It could say that the PR consists of multiple commits and the commit message needs to be summarised when merging, and suggest that if the submitter is willing, they could squash and summarise the message themselves. I don't want to give the impression we're insisting the submitter squash, rather we're pointing out there's an additional step needed and the submitter can help by doing that step if they wish.
Yeah, that would probably be useful for regular committers. Paul

On 8/31/2017 3:27 PM, Brett Cannon wrote:
On Wed, 30 Aug 2017 at 02:56 Paul Moore <p.f.moore@gmail.com
My experience is different.
So you would want a comment when the PR reaches "awaiting merge" with instructions requesting the author do their own squash commit to
My impression is that this makes the individual commits disappear.
simplify the message for us?
Not me. -- Terry Jan Reedy

On 30 August 2017 at 19:39, Antoine Pitrou <solipsis@pitrou.net> wrote:
Not while we're pushing the "squash & merge" button directly, as there's no opportunity for any automated hooks to run at that point :( More options open up if the actual commit is being handled by a bot, but even that would still depend on us providing an updated commit message via some mechanism. Cheers, Nick. -- Nick Coghlan | ncoghlan@gmail.com | Brisbane, Australia

On 30 August 2017 at 10:48, Nick Coghlan <ncoghlan@gmail.com> wrote:
We could have a check for PRs that contain multiple commits. The check would flag that the commit message needs manual editing before merging, and would act as a prompt for submitters who are willing to do so to squash their PRs themselves. Personally, I'd prefer that as, apart from admin activities like making sure the PR and issue number references are present and correct, I think the original submitter has a better chance of properly summarising the PR in the merged commit message anyway. Paul

On Wed, 30 Aug 2017 at 02:56 Paul Moore <p.f.moore@gmail.com> wrote:
So you would want a comment when the PR reaches "awaiting merge" with instructions requesting the author do their own squash commit to simplify the message for us? There's also https://github.com/python/bedevere/issues/14 to help post-commit remind core devs when they forgot to do something. While it would be too late to fix a problem, hopefully regular reminders would still be helpful.

On 31 August 2017 at 20:27, Brett Cannon <brett@python.org> wrote:
That would work. It could say that the PR consists of multiple commits and the commit message needs to be summarised when merging, and suggest that if the submitter is willing, they could squash and summarise the message themselves. I don't want to give the impression we're insisting the submitter squash, rather we're pointing out there's an additional step needed and the submitter can help by doing that step if they wish.
Yeah, that would probably be useful for regular committers. Paul

On 8/31/2017 3:27 PM, Brett Cannon wrote:
On Wed, 30 Aug 2017 at 02:56 Paul Moore <p.f.moore@gmail.com
My experience is different.
So you would want a comment when the PR reaches "awaiting merge" with instructions requesting the author do their own squash commit to
My impression is that this makes the individual commits disappear.
simplify the message for us?
Not me. -- Terry Jan Reedy
participants (6)
-
Antoine Pitrou
-
Brett Cannon
-
Nick Coghlan
-
Paul Moore
-
Serhiy Storchaka
-
Terry Reedy