What's the status on "Squash and merge"?

I'd like to merge the PR at https://github.com/apache/netbeans/pull/2410 , but 
I can't remember how to verify that "Squash and merge" won't put in bogus 
author information.

-- Eirik

-----Original Message-----
From: Neil C Smith <neilcsm...@apache.org> 
Sent: Sunday, August 9, 2020 8:43 AM
To: dev <dev@netbeans.apache.org>
Subject: Re: Merging on github (aka Squash and Merge is broken)

On Thu, 12 Dec 2019 at 20:41, Matthias Bläsing <mblaes...@doppel-helix.eu> 
wrote:
> 14. Switch back to master
>
>    git checkout master
>
> 15. Merge (commit hash from above):
>
>    git merge c20b63503c98
>
> 16. Push
>
>    git push
>
>
> Do this multiple times with test sets, then its _very_ easy. The 
> alternative is of course to ask the author to do the squash himself, 
> he can update the PR (requires a forced push)

Just following up on this.  I've had a few of these come up during
12.1 so far.  I'm also not entirely a fan of bypassing the PR system or direct 
pushes to upstream master (I have push-url set to disabled by default).

As well as the author squashing the PR, it is also possible for any committer 
to force push the PR branch (by default anyway) if the author cannot for some 
reason.  I did this with eg.
https://github.com/apache/netbeans/pull/2023  And I think all is fine with the 
git history on this.

Matthias, could you have a quick check on that?  I used your instructions 
roughly up to where to push to.  But this has the benefit of linking back to 
original PR / closing, allowing the author to check over, and running CI tests 
again before merging.  Thoughts on pros / cons of updating instructions to 
recommend this?

Thanks and best wishes,

Neil

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@netbeans.apache.org
For additional commands, e-mail: dev-h...@netbeans.apache.org

For further information about the NetBeans mailing lists, visit:
https://cwiki.apache.org/confluence/display/NETBEANS/Mailing+lists




---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@netbeans.apache.org
For additional commands, e-mail: dev-h...@netbeans.apache.org

For further information about the NetBeans mailing lists, visit:
https://cwiki.apache.org/confluence/display/NETBEANS/Mailing+lists



Reply via email to