In our meeting today we discussed the need for more patch/commit reviewing and that there should be rules in place as to what should require a review before being pushed into master. These were some suggestions: Any commit that: A. Is not a fix or a feature planned for the next release B. Is greater than 50 lines C. Is in an area of the code base that has more than one developer contributing
Other suggestions were to require branch pushing for CI certification before pushing to master. Please discuss your ideas here so we can reach an agreement or what our process for requiring code review should be. ------------------------------------------------------------------------------ Check out the vibrant tech community on one of the world's most engaging tech sites, Slashdot.org! http://sdm.link/slashdot _______________________________________________ enlightenment-devel mailing list enlightenment-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/enlightenment-devel
