Hi Jark There is a conversation about this here: https://github.community/t5/How-to-use-Git-and-GitHub/Authorship-of-merge-commits-made-by-Github-Apps-changed/td-p/48797 I think GitHub will fix it soon, it is a bug, not a feature :).
Jingsong Li <jingsongl...@gmail.com> 于2020年3月5日周四 下午8:32写道： > Thanks for deep investigation. > > +1 to disable "Squash and merge" button now. > But I think this is a very serious problem, It affects too many GitHub > workers. Github should deal with it quickly? > > Best, > Jingsong Lee > > On Thu, Mar 5, 2020 at 7:21 PM Xingbo Huang <hxbks...@gmail.com> wrote: > > > Hi Jark, > > > > Thanks for bringing up this discussion. Good catch. Agree that we can > > disable "Squash and merge"(also the other buttons) for now. > > > > There is a guideline on how to do that in > > > > > https://help.github.com/en/github/administering-a-repository/configuring-commit-squashing-for-pull-requests > > . > > > > Best, > > Xingbo > > > > Jark Wu <imj...@gmail.com> 于2020年3月5日周四 下午6:42写道： > > > > > Hi everyone, > > > > > > We just noticed that everytime a pull request gets merged with the > > "Squash > > > and merge" button, > > > GitHub drops the original authorship information and changes "authored" > > to > > > whoever merged the PR. > > > > > > We found this happened in #11102 [1] and #11302 [2]. It seems that it > is > > a > > > long outstanding issue > > > and GitHub is aware of it but doesn't make an attempt to fix it [3][4]. > > > > > > Before this behavior, "authored" is the original author and > "committed" > > is > > > the one who merged the PR, > > > which was pretty good to record the contributor's contribution and the > > > committed information. > > > > > > From the perspective of contributors, it’s really frustrated if their > > > authorship information gets lost. > > > Considering we don't know when GitHub will fix it, I propose to disable > > > "Squash and merge" button > > > (and also "Rebase and merge" button) before it is fixed. > > > > > > However, I'm not sure how to disable it. Can it be disabled by GitHub > UI > > if > > > who has administrator permission? > > > Or .asf.yaml [5] is the right way? > > > > > > What do you think? > > > > > > Best, > > > Jark > > > > > > [1]: https://github.com/apache/flink/pull/11102 > > > [2]: https://github.com/apache/flink/pull/11302 > > > [3]: > https://github.com/chdsbd/kodiak/issues/300#issuecomment-595016815 > > > [4]: https://github.com/isaacs/github/issues/1750 > > > [5]: > > > > > > > > > https://cwiki.apache.org/confluence/display/INFRA/.asf.yaml+features+for+git+repositories#id-.asf.yamlfeaturesforgitrepositories-Mergebuttons > > > > > > > > -- > Best, Jingsong Lee >
