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
Since your branches are rebased with upstream master, merges are
fast-forward. This also ensures that conflicts will be solved by
whoever contributed the patch, easing the burden on the person
appointed as the "merge driver".
Note that this does not require anyone to actually go through the tool
to perform a commit. One can simply push changes to the upstream
master, bypassing review altogether. This can of course be a problem
and still lead to "woooo typo--" commits, but if we'll perform reviews
before committing things to the master repository, these shouldn't be
(too) necessary anyway -- and, if they are, the bureaucracy overhead
is zero.
In any case: the review tool is amazing. You can select blocks to
comment, and at the very end choose if you approve or not, and make
some overall remarks. It even copies all your previous comments near
the bottom of the page so that you don't have to scroll. A nice touch
is: you can begin reviewing, close the browser, and your changes will
be there awaiting your submission.
The task stuff is nice as well. I like that things are integrated: a
task might depend on a review and vice-versa.
The wiki is also pretty nice. Also of note: the Wiki syntax is, like
on Trac, available almost everywhere where there is a multiline text
entry. For larger entries, there is even a as-you-type preview so you
don't have to press any 'Preview' button, unlike Trac.
There is even a Pastebin there, integrated with their command-line
tool, arcanist. The nicest thing about it, of course, out of the box
syntax highlight for Brainfuck snippets. There are various of these
easter eggs as well, which can sadly be turned off by saying that your
installation is "serious business". I do not recommend that, though.
Ah, of importance as well: the application itself is very snappy. Way
faster than Trac, and without awkward things like having to restart
the web server when a new milestone is created.
Cheers,
Leandro
------------------------------------------------------------------------------
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
