Branch: refs/heads/dev Home: https://github.com/galaxyproject/galaxy Commit: ce6cd03f6c7f020a8127025c49224f1bb920bc18 https://github.com/galaxyproject/galaxy/commit/ce6cd03f6c7f020a8127025c49224... Author: John Chilton <jmchilton@gmail.com> Date: 2015-12-10 (Thu, 10 Dec 2015) Changed paths: A .ci/check_controller.sh A .ci/check_mako.sh M tox.ini Log Message: ----------- Enforce metrics related to moving toward modern client infrastructure. Having a CI enforced blacklist and then a whitelist of files failing to meet style guidelines (as well as expanding list of rules enforced) may have helped create incentive to rework Galaxy toward a more compliant and uniform Python styling. We briefly discussed the possibility of doing this with other team code base objectives on a conference call - in particular to help the transition toward replacing all of the web controllers with API controllers and eliminating the use of Python mako files for rendering web content and replacing it with client side templates. To do this I have created two rough metrics - the number of lines of web controller code in the Galaxy app and the number of mako files in the templates directory. The hope is that people feel ... compelled to lower these numbers. While these metrics are admittedly rough, they do also encourage de-duplication of mako templates and transition of API code into managers (in addition to elimination of functions) both of these secondary incentives are also good things and should be project objectives. Commit: a7e60eae2c2721e7d4377112b7aaceb8424fd3da https://github.com/galaxyproject/galaxy/commit/a7e60eae2c2721e7d4377112b7aac... Author: Eric Rasche <rasche.eric@gmail.com> Date: 2015-12-11 (Fri, 11 Dec 2015) Changed paths: A .ci/check_controller.sh A .ci/check_mako.sh M tox.ini Log Message: ----------- Merge pull request #1292 from jmchilton/client_incentives Enforce metrics related to moving toward modern client infrastructure. Compare: https://github.com/galaxyproject/galaxy/compare/eb4b8e716d26...a7e60eae2c27