mbien commented on PR #147: URL: https://github.com/apache/netbeans-mavenutils-nbm-maven-plugin/pull/147#issuecomment-1956673733
@charphi cool thanks. Yes we know that we can let github squash during integration. This caused problems in past, author information can get lost etc and it add opportunities for more mistakes, e.g selecting the wrong option and then merging 20 commits instead of squashing - which happened too. We simply ask squash the PR to the desired number of commits as last step post review. We can then merge the commits unmodified without changing anything. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected] For further information about the NetBeans mailing lists, visit: https://cwiki.apache.org/confluence/display/NETBEANS/Mailing+lists
