Review Request: BMR: activity runner
Aaron J. Seigo
aseigo at kde.org
Wed Feb 2 23:42:03 CET 2011
On Wednesday, February 2, 2011, you wrote:
> a test branch merge request (BMR :) for aseigo/activityrunner
i'd like to propose that we use review board for merge requests, particularly
as a way to signal that "this branch is ready!" for new features. it's a
familiar tool to us all, if a bit different with git.
to generate this one i did:
git checkout aseigo/activityrunner
git merge master
post-review --parent=master --tracking-
for more info on using post-review with kde's git infrastructure. post-review
needs some work so it can understand git shorcut urls :) anyways ...
in this proposal, whenever we wish to merge a feature branch into master (or,
later, into our integration branch) we would create such a MR in reviewboard.
these RB entries should have topics that start with "MR:" so we know it's an
actual merge request rather than something else.
even if you merge the branch yourself, i think we should create one of these
so that they can be tracked. once it has been merged into master, then it can
be closed. if/when we get an integration branch, i think this can become
particularly important for the person responsible for merging integration into
esp if we end up with an "features always welcome in integration branch"
policy, then it will be crucial to be able to keep track of what has been
merged during freeze periods in the release schedule. i'm not sure RB will be
the best place for this yet, and i hope to discuss this topic with people
wiser with git than me at the kdelibs git workflow meeting.
Aaron J. Seigo
humru othro a kohnu se
GPG Fingerprint: 8B8B 2209 0C6F 7C47 B1EA EE75 D6B7 2EB1 A7F1 DB43
KDE core developer sponsored by Qt Development Frameworks
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Size: 198 bytes
Desc: This is a digitally signed message part.
Url : http://mail.kde.org/pipermail/plasma-devel/attachments/20110202/7c8b3e7e/attachment.sig
More information about the Plasma-devel