On Sun, 08 Apr 2012 13:32:07 -0700, Platonides <Platonides(a)gmail.com>
wrote:
You're a brave soul!
Can we get a labs project for testing that in real time?
I'm not sure about the personal public remotes. If we're going to
continue with our way of contribution, we would have continuous bases.
OTOH it looks cool to be able to work A1--A2--B1--B2 without creating
new branches at each step.
Well, you can work that way. Though I think gareth will
still have similar
handling where using a branch name makes the review system automatically
guess what name to use for the topic/review branch.
It also shields you from some unnecessary merge conflicts.
The review-heads stuffs is perhaps too complicated. I
think it'd be
simpler to base in the well-known monotonic-increased ids.
So the review-queue item 1024 is at refs/review-heads/1024 where you
pull the latest item (or the full set?), and each forked head for that
review has refs/review-heads/1024/1, refs/review-heads/1024/2...
Plus the aliasing of 1024 to Voice-to-wikitext-feature.
Reuse of old names might be confusing. Maybe if a name is reused it
should get named Feature_v2 and the original one automatically named
Feature_v1. The mere presence of v2 indicates that there was a prior one.
Hmm... I'll consider it. But this project's focus has been trying to
embrace the git way of doing things; since some of gerrit's issues stem
from ignoring that way of doing things.
- The project
needs a good database system. I copied our database
classes in but never got to using them. I'm isolating all database stuff
into some model classes so different database handing can be swapped in.
Anyone who feels up to it can adapt our database code to work as a
framework for the review system.
I think it would be easier with our classes. Too much fetch()s in
ProjectModel.php
Someone would need to adapt the classes to work outside MW.
I started using Mysqli and prepared statements since it's an easy way to
get the database stuff out of the way right away.
- Right now
I'm implementing git handling using proc_open to interact
with git's porcelain and plumbing commands. Anyone who feels up to the
task is free to implement a PHP extension for interfacing with git we
can swap over to using.
Please, remove the usage of __call() there. Make a different function
for each one, even if they're going to be dummy ones right now.
This way we can easily replace them with real implementations instead of
wondering which ones are called by the magic.
Drop ShellGit entirely and make Git the only class?
--
~Daniel Friesen (Dantman, Nadir-Seen-Fire) [
http://daniel.friesen.name]