+1 on the PR titles for squashes. That would help a lot! For the auto merge, is that setting off by default per PR? Reading the docs they say that you can enable/disable for each PR but not what the default is (at least not that I saw). If we can have it default to manual, I’d feel more comfortable in avoiding situations where PRs are merged before the author is ready if they weren’t expecting it. +1 if that’s the case.
Elizabeth > On Aug 29, 2022, at 9:28 PM, Tai Dupree <tdupree...@gmail.com> wrote: > > Hi Team, > > There are a couple git features that I'd like to request apache infra to > enable. These settings are not supported by the asf.yaml configuration. I'd > like to request the following feature be enabled: > > - auto merge > <https://docs.github.com/en/repositories/configuring-branches-and-merges-in-your-repository/configuring-pull-request-merges/managing-auto-merge-for-pull-requests-in-your-repository>, > should make merges faster as CI can take long but having github merge once > it's green would be a nice quality of life improvement. > > - Default to pull request titles for squash commits > <https://docs.github.com/en/repositories/configuring-branches-and-merges-in-your-repository/configuring-pull-request-merges/configuring-commit-squashing-for-pull-requests>, > for some reason when there's only 1 commit in the PR, github uses the > commit message instead of the PR title. However, if the contains more than > 1 commit, the PR title is used. This should make our git history better > since we have rules around PR titles. > > Please respond with any concerns, comments, or +1s. If there are no > concerns over the next few days, I'll go ahead and open the infra ticket. > > Best, > -- > -Tai Dupree
