On Mon, 10 Sep 2012 16:43:44 -0300 Leandro Pereira <lean...@profusion.mobi> said:
> 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. i poked around a bit and phabricator seems like the best option out there that i've seen. its default look.. well.. that's a matter of taste, but it's all fixable. :) only thing i see here is that you are expecting a workflow of everyone working in branches and asking for reviews. i honestly don't see that happening. well not for everything and everyone. what i have been hoping for is a review system for CASUAL contributors. those that are not really familiar with everything so they need to be looked over. that is where the win comes in. :) -- ------------- Codito, ergo sum - "I code, therefore I am" -------------- The Rasterman (Carsten Haitzler) ras...@rasterman.com ------------------------------------------------------------------------------ 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
