Changes

Jump to navigation Jump to search
no edit summary
Line 3: Line 3:  
* Reviewer notes should cover only logic of patch, other things will be caught on pre-commit level (if hook lacks of some checks, it shouldn't be subject for review process any way, and hooks should be fixed)
 
* Reviewer notes should cover only logic of patch, other things will be caught on pre-commit level (if hook lacks of some checks, it shouldn't be subject for review process any way, and hooks should be fixed)
   −
* Submiting patches via email sounds pretty obvious and comfortable since glucose uses git.<br>But we need to have some kind of centralization to simplify releasing workflows and do not enforce people to setup local environment if they used to track patches in web browser.<br> Possible options are:
+
* Submiting patches via email sounds pretty obvious and comfortable since glucose uses git.<br>But we need to have some kind of centralization to simplify releasing workflows and do not enforce people to setup local environment if they used to track patches in web browser.<br> Possible options are, vote for particular option on [http://idea.sugarlabs.org/drupal5/ideatorrent/idea/36/]:
 
** We dont need such centralization at all, this is just remains from previous workflow
 
** We dont need such centralization at all, this is just remains from previous workflow
 
*** ''no comments :)''
 
*** ''no comments :)''

Navigation menu