for large PRs, we tend to have lot's of commits for review, and tend to manually squash/rebase before final pushOn Tue, Dec 13, 2016 at 7:20 PM, Wojciech Trocki <wtrocki@redhat.com> wrote:Thanks Sebi. I wasn't aware of this document.It actually matches my suggestion.Recently github added 2 different options for PR's so suggestion here is to also use this options more to automate merges.
Git workflow should not be affected.On Tue, Dec 13, 2016 at 5:54 PM, Sebastien Blanc <scm.blanc@gmail.com> wrote:Why don't keep following what is defined here https://aerogear.org/docs/guides/GitHubWorkflow/ ?On Tue, Dec 13, 2016 at 6:44 PM, Wojciech Trocki <wtrocki@redhat.com> wrote:______________________________TL;DR - Suggested options:
EXPLANATION:- Default option used for most of the cases- Use when you want to integrate changes with multiple commitsOn Tue, Dec 13, 2016 at 5:19 PM, Wojciech Trocki <wtrocki@redhat.com> wrote:Hi Aerogear Developers :)Do you guys think that we can use github squash or merge option as default action for aerogear digger repos (or even every aerogear repository)?Aerogear contribution guide doesn't state how tickets are reviewed etc. but I see that most of the opensource projects use squashing option after merging to master - this would still allow us to push as many commits as we want. Squashing would be performed before merge to master on github. This would make our git logs clean and would simplify releases etc.I do not want to start some holly war here, but I would just like to keep things clean and avoid some commits to repo without actual meaning or simply incomplete. When checking changes on the master multiple commits can make things difficult. It would be hard to provide some release notes basing on that etc.
Regards--Wojciech TrockiSoftware Engineer, Red Hat Mobile--Wojciech TrockiSoftware Engineer, Red Hat Mobile_________________
aerogear-dev mailing list
aerogear-dev@lists.jboss.org
https://lists.jboss.org/mailman/listinfo/aerogear-dev
_______________________________________________
aerogear-dev mailing list
aerogear-dev@lists.jboss.org
https://lists.jboss.org/mailman/listinfo/aerogear-dev --Wojciech TrockiSoftware Engineer, Red Hat Mobile
_______________________________________________
aerogear-dev mailing list
aerogear-dev@lists.jboss.org
https://lists.jboss.org/mailman/listinfo/aerogear-dev --Matthias Wessendorf
blog: http://matthiaswessendorf.wordpress.com/
twitter: http://twitter.com/mwessendorf
_______________________________________________
aerogear-dev mailing list
aerogear-dev@lists.jboss.org
https://lists.jboss.org/mailman/listinfo/aerogear-dev