patch pilot report

Jonathan Riddell jriddell at ubuntu.com
Mon Aug 22 10:07:01 UTC 2011


Shannon Weyrick's patch to merge on large files.  We got this finished off and merged in.
https://code.launchpad.net/~weyrick/bzr/54624-warn-on-large-files/+merge/70691

"bzr push -q now does not print name of pushed location" tidied up and merged
https://code.launchpad.net/~asc/bzr/bzrpushmessagesuppress/+merge/66909

"Add missing config store and stacks for control.conf and remote branches." approved with docstrings added
https://code.launchpad.net/~vila/bzr/missing-stacks/+merge/71918

"config.LocationMatcher properly excludes unrelated sections" approved
https://code.launchpad.net/~vila/bzr/829237-location-matcher-misses/+merge/72151

'Add a "bzr branches" command.' approved
https://code.launchpad.net/~jelmer/bzr/cmd-branches/+merge/71577

and my own ones..

"Document gpgme usage in comments." and 'Report commits signed with expired keys in "verify-signatures".'
finally merged after some battling with PQM
https://code.launchpad.net/~jr/bzr/804253-comment-gpgme-verify-usage/+merge/70337
https://code.launchpad.net/~jr/bzr/804254-expired-keys/+merge/70419

'qverify-signatures: Report commits signed with expired keys in "verify-signatures"'
https://code.launchpad.net/~jr/qbzr/verify-signatures-expired/+merge/70422

With patch piloting I struggle that I'm not qualified to review many
patches, the last thing I want to do is approve a patch only to find
it contains some obvious problem I overlooked.  Also many merge
proposals have ongoing conversations and it seems those who have already
commented are best qualified to take it to approval.

On patch pilot duty this week is John.

Jonathan



More information about the bazaar mailing list