Why "pullRequestAutoMerge" forces a squash merge? #1841
Closed
PSchwamborn-AINO
started this conversation in
Ideas
Replies: 1 comment 1 reply
-
Thanks for the feedback! I don't think there's any particular reason squash commits are needed so we could add a property to the settings that allows you to configure this |
Beta Was this translation helpful? Give feedback.
1 reply
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Uh oh!
There was an error while loading. Please reload this page.
-
I wanted to use the "pullRequestAutoMerge" property for our Update AL-Go System Files.

We have a lot of repositories and do not want to allow direct commits. So this setting is very useful for us.
Now I am wondering why this property forces a squash commit:
https://github.com/microsoft/AL-Go-Actions/blob/91d88b04dad77542fb8ac5a7a3cebee07e71952f/AL-Go-Helper.ps1#L1489
The problem is that we do not want to allow squash merged for our repositories. So, they are deactivated, and the action throws an error:
It's annoying that he created the pull request anyway, but I think that's a different issue...
So a new property that allows us to define the merge method would be very useful.
Or is there a specific reason why a squash merge is necessary here?
Beta Was this translation helpful? Give feedback.
All reactions