Merge lp:~zyga/checkbox/fix-1438643 into lp:checkbox

Proposed by Zygmunt Krynicki
Status: Merged
Approved by: Sylvain Pineau
Approved revision: 3634
Merged at revision: 3634
Proposed branch: lp:~zyga/checkbox/fix-1438643
Merge into: lp:checkbox
Diff against target: 12 lines (+1/-1)
1 file modified
plainbox/plainbox/provider_manager.py (+1/-1)
To merge this branch: bzr merge lp:~zyga/checkbox/fix-1438643
Reviewer Review Type Date Requested Status
Sylvain Pineau Approve
Review via email: mp+254729@code.launchpad.net

Description of the change

This branch fixes a trivial but annoying bug in plainbox provider manager.

To post a comment you must log in.
Revision history for this message
Sylvain Pineau (sylvain-pineau) wrote :

+1 for store_true!

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'plainbox/plainbox/provider_manager.py'
2--- plainbox/plainbox/provider_manager.py 2015-03-30 16:46:19 +0000
3+++ plainbox/plainbox/provider_manager.py 2015-03-31 11:18:36 +0000
4@@ -1174,7 +1174,7 @@
5 help=_("don't actually do anything"))
6 group = parser.add_argument_group(title=_("actions to perform"))
7 group.add_argument(
8- "--dont-update-pot", default=False, action="store_false",
9+ "--dont-update-pot", default=False, action="store_true",
10 help=_("do not update the translation template"))
11 group.add_argument(
12 "--dont-merge-po", default=False, action="store_true",

Subscribers

People subscribed via source and target branches