Hi, +1 with this proposal, thx Otavio
Regards, Nicolas ________________________________ From: Otavio Rodolfo Piske <angusyo...@gmail.com> Sent: Wednesday, July 19, 2023 10:38 To: dev <dev@camel.apache.org> Subject: Proposal for improving the way we collaborate on code improvements Hi folks, I am writing this proposed changes based on the feedback received on the previous email where I shared some thoughts/concerns about improving our contribution practices (see the email entitled "Some thoughts on bisecting Camel and our git commit practices" for details). This is mostly aimed at the Camel Core sub-project (potentially, we can use it later for Camel Spring Boot, Camel Kafka Connector, and Camel Karaf). My understanding is that there are 2 practices we can start using that can improve the quality of our commits and ensure our commits are compilable and easy to bisect. 1. All code changes should done via Pull Requests: The idea is to reduce the incidence of non-compilable commits by ensuring that any code goes through Github compilation and testing. This helps both the contributors, by ensuring their contributions are complete and testable as well as the rest of the community, by ensuring that the code doesn't break the tree for others 2. Squashing as the default way of merging code The idea is to make sure that the contribution is seen as a single unit of change, including the build result of the contribution which reflects the final state of the branch. There may be a few cases where going through this process might be a bit too much. For instance: changing the code for CI automation, modifying code that is clearly not compilable (i.e.: readme files or other text files that are not documentation for the website, but provide support for developers, etc). I suggest we use the same approach for these files, but don't necessarily make ourselves bound to wait for the GH CI to run before merge. Please, what do you think about this? If there's agreement, how could we possibly start using this process? Kind regards -- Otavio R. Piske https://urldefense.com/v3/__http://orpiske.net__;!!CiXD_PY!QeXANw1BXPMNS_EaRwNAK4JdJzrM_7IffR1DL_aOV41Qhyy74RSrEhNcYu_fGFwAZKGJXnqV-sEWdgL9vyc$ As a recipient of an email from the Talend Group, your personal data will be processed by our systems. Please see our Privacy Notice <https://www.talend.com/privacy-policy/> for more information about our collection and use of your personal information, our security practices, and your data protection rights, including any rights you may have to object to automated-decision making or profiling we use to analyze support or marketing related communications. To manage or discontinue promotional communications, use the communication preferences portal<https://info.talend.com/emailpreferencesen.html>. To exercise your data protection rights, use the privacy request form<https://talend.my.onetrust.com/webform/ef906c5a-de41-4ea0-ba73-96c079cdd15a/b191c71d-f3cb-4a42-9815-0c3ca021704cl>. Contact us here <https://www.talend.com/contact/> or by mail to either of our co-headquarters: Talend, Inc.: 400 South El Camino Real, Ste 1400, San Mateo, CA 94402; Talend SAS: 5/7 rue Salomon De Rothschild, 92150 Suresnes, France
