On 12-09-10 16:43, Leandro Pereira wrote:
> Luis,
> 
> On Mon, Sep 10, 2012 at 8:40 AM, Luis Felipe Strano Moraes
> <luis.str...@gmail.com> wrote:
> > Honestly, given the amount of people working on Phabricator, I would
> > much rather go forward with it (if it does indeed cover everything
> > that is needed in our case, which seems to be the case). InDefero
> > looked like it had much less frequent commits, and from the link above
> > it seems it is no longer the priority for the main author.
> 
> After using Phabricator for some weeks, I feel it is pretty stable and
> easy to use. While it encourages a clean and organized workflow it
> does not enforce it, which might ease the adoption process or create a
> mess if people is not careful enough.
> 
> The basic workflow is as follows:
>  - Your master is always rebased on upstream master
>  - For each bugfix/feature, you create a local branch, rebased on your master
>  - Hack and commit as needed; then ask for review
>  - If reviewed OK, you land this bugfix/feature branch; someone merges
> with your master to the upstream master
>  - If reviewed negatively, you perform all requested changes, commit
> as usual, and then request a new review

That looks like a sane workflow (i would say “à la svn”).

  Does rebasing mess up with review? That's the case with gerrit and
once a commit has been put to review, it's better to rebase it only just
before merging or diffs are screwed up.
  Does phabricator makes it easy to review branches or is it only great
to review one commit at a time?

> […] The nicest thing about it, of course, out of the box
> syntax highlight for Brainfuck snippets.

\o/ That's great!
-- 
Boris Faure
Pointer Arithmetician

------------------------------------------------------------------------------
Live Security Virtual Conference
Exclusive live event will cover all the ways today's security and 
threat landscape has changed and how IT managers can respond. Discussions 
will include endpoint security, mobile security and the latest in malware 
threats. http://www.accelacomm.com/jaw/sfrnl04242012/114/50122263/
_______________________________________________
enlightenment-devel mailing list
enlightenment-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/enlightenment-devel

Reply via email to