On Thu, Jul 10, 2014 at 1:32 PM, Stefan Schmidt <ste...@datenfreihafen.org> wrote: > Hello. > > On Thu, 2014-07-10 at 13:10, Cedric BAIL wrote: >> On Thu, Jul 10, 2014 at 12:12 PM, Stefan Schmidt >> <ste...@datenfreihafen.org> wrote: >> > Hello. >> > >> > On Thu, 2014-07-10 at 01:44, ms wrote: >> >> cedric pushed a commit to branch master. >> >> >> >> http://git.enlightenment.org/core/efl.git/commit/?id=ce8d67922fcd9d18b887f17c29cdd5749eafecd0 >> >> >> >> commit ce8d67922fcd9d18b887f17c29cdd5749eafecd0 >> >> Author: ms <ms.g...@samsung.com> >> >> Date: Thu Jul 10 10:39:06 2014 +0200 >> >> >> >> test_test >> >> >> >> Reviewers: kimcinoo >> >> >> >> Reviewed By: kimcinoo >> >> >> >> Subscribers: kimcinoo, cedric >> >> >> >> Differential Revision: https://phab.enlightenment.org/D1153 >> > >> > Is this really what we expect from a commit message? Sure its a test >> > for finding the test but "test_test" is not what I expect. >> >> Me to, got tricked by arcanist again on that one. Sorry. Just looked >> at the code... Will pay more attention at the commit message next >> time. > > What did phab do this time? > > We already have two pitfalls with the review system: > 1) Author name gets replaces with phab name
Oh, didn't know that one. > 2) Whit space changes get ignored That one was fixed. > Both already hit us before. Is there another ptifall about the commit > messages? Me not looking at it. -- Cedric BAIL ------------------------------------------------------------------------------ Open source business process management suite built on Java and Eclipse Turn processes into business applications with Bonita BPM Community Edition Quickly connect people, data, and systems into organized workflows Winner of BOSSIE, CODIE, OW2 and Gartner awards http://p.sf.net/sfu/Bonitasoft _______________________________________________ enlightenment-devel mailing list enlightenment-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/enlightenment-devel
