El sáb, 19-03-2011 a las 13:43 +0100, Hans De Bisschop escribió: > Hi all, > > Just spent some time writing a new script that enables us to get > notified by mail of new changesets on any of the Chamilo 2 > repositories. The format of said mails can be seen below. Compared to > the "old" script format on Google Code I've added direct links to > comparisons for modified files.
This is great. Could you add an excerpt of the modified code? I know I'm probably the only guy doing this (but I don't consider I *should* be the only one - this is just basic peer review), but I usually review the type of code change, and clicking on a link doesn't allow for an "unplugged" review (and highly depends on the availability of bitbucket and an internet connection at that point). Funnily enough, I spend most of my unplugged working time to review code from others to ensure high quality of the commits. > I'll be adding the post-hook to all repositories over the next few > days. Might it be a good idea to have a seperate mailing list (the > irony is not lost upon me) for all these messages to arrive on? Would > prefer that they don't clutter up the regular list. I can create that one for you and highly agree (for the unplugged reasons mentioned above). How should it be named? dev2-commits? > Best regards, > Hans > > -------- Original Message -------- > Subject: > C2M > chamilo-ext-repo-matterhorn-dev: > [Scaramanga] Testing the new > push-hook script. > Date: > Sat, 19 Mar 2011 13:41:18 +0100 > From: > Chamilo 2 AT Bitbucket > <dev@lists.chamilo.org> > Reply-To: > <cham...@googlegroups.com> > To: > Chamilo 2 Notifications > <cham...@googlegroups.com> > > > Repository: chamilo-ext-repo-matterhorn-dev > Revision: 64 > Node: 9cde15af93df > Link: > https://bitbucket.org/chamilo/chamilo-ext-repo-matterhorn-dev/changeset/9cde15af93df > > Author: Scaramanga > Date: 2011-03-19 13:39:55 > Pushed by: ScaraMcDuck > > Log Message: > ------------ > Testing the new push-hook script. > > Added Paths: > ------------ > php/temp/test.xml > > Modified Paths: > --------------- > php/settings/settings_matterhorn.xml > (https://bitbucket.org/chamilo/chamilo-ext-repo-matterhorn-dev/diff/php/settings/settings_matterhorn.xml?diff2=9cde15af93df&diff1=99c990e56d6b) > > > _______________________________________________ > Dev mailing list > Dev@lists.chamilo.org > http://lists.chamilo.org/listinfo/dev _______________________________________________ Dev mailing list Dev@lists.chamilo.org http://lists.chamilo.org/listinfo/dev
