[MERGE] HACKING transformed into Bazaar Developer Guide
Ian Clatworthy
ian.clatworthy at internode.on.net
Wed May 2 05:54:37 BST 2007
Aaron Bentley wrote:
> Some of this TDD stuff is about testing in general, rather than Test
> Driven Development. While we can recommend TDD, what we require is
> proper testing. So I think we need a different headline.
>
>
Fixed as discussed on IRC.
> SIGQUIT is not SIGINT; SIGINT is caused by CTRL-C. CTRL-\ is required
> to produce SIGQUIT, not CTRL-C.
>
>
I think this was mainly a formatting issue with a \ at the end of line
confusing the output. Corrected.
> There is no such thing as a bzr changeset.
>
> Also, I recommend using merge directives rather than sending a diff+branch.
>
>
> Please do *NOT* put [PATCH] or [MERGE] in the subject line if you don't
> want it to be merged.
>
>
Section improved to address all of these.
>> +If you read a patch please reply and say so. We can use a numeric scale
>> +of -1, -0, +0, +1, meaning respectively "really don't want it in current
>> +form", "somewhat uncomfortable", "ok with me", and "please put it in".
>> +Anyone can "vote". (It's not really voting, just a terse expression.)
>> +
>> +If something gets say two +1 votes from core reviewers, and no
>> +vetos, then it's OK to come in. Any of the core developers can bring it
>> +into their integration branch, which I'll merge regularly. (If you do
>> +so, please reply and say so.)
>>
>
> ^^^^ This is out of date, now that BB has formalized voting.
>
>
Using the BB help as I guide, I've updated this text. I had to clarify
the meanings a bit though because 0+ and 1+ don't really translate to
"ok by me" and "put it in" - not everyone has access to do the latter
and I guess that was implied in earlier days.
Latest bundle attached. It includes the fix Alexander requested as well.
Once again, the net HTML result is attached for those without rst2html
installed.
Ian C.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: HACKING-as-developer-guide3.patch
Type: text/x-patch
Size: 107405 bytes
Desc: not available
Url : https://lists.ubuntu.com/archives/bazaar/attachments/20070502/e6789197/attachment-0001.bin
-------------- next part --------------
An HTML attachment was scrubbed...
URL: https://lists.ubuntu.com/archives/bazaar/attachments/20070502/e6789197/attachment-0001.html
More information about the bazaar
mailing list