Merge lp://staging/~jelmer/lptools/python3 into lp://staging/lptools

Proposed by Jelmer Vernooij
Status: Merged
Merged at revision: 48
Proposed branch: lp://staging/~jelmer/lptools/python3
Merge into: lp://staging/lptools
Diff against target: 888 lines (+134/-132)
20 files modified
bin/lp-attach (+9/-9)
bin/lp-bug-dupe-properties (+6/-6)
bin/lp-capture-bug-counts (+5/-5)
bin/lp-check-membership (+4/-4)
bin/lp-force-branch-mirror (+4/-4)
bin/lp-get-branches (+7/-7)
bin/lp-grab-attachments (+2/-2)
bin/lp-list-bugs (+8/-9)
bin/lp-milestone2ical (+4/-4)
bin/lp-milestones (+2/-2)
bin/lp-project (+1/-1)
bin/lp-project-upload (+25/-23)
bin/lp-recipe-status (+7/-7)
bin/lp-remove-team-members (+5/-5)
bin/lp-review-list (+8/-8)
bin/lp-review-notifier (+8/-8)
bin/lp-set-dup (+16/-16)
bin/lp-shell (+8/-8)
lptools/config.py (+1/-1)
setup.py (+4/-3)
To merge this branch: bzr merge lp://staging/~jelmer/lptools/python3
Reviewer Review Type Date Requested Status
Guruprasad Approve
Review via email: mp+436166@code.staging.launchpad.net

Description of the change

Port to Python3

To post a comment you must log in.
Revision history for this message
Guruprasad (lgp171188) wrote :

LGTM 👍 Are these the same changes that are shipping in the lptools Debian/Ubuntu packages already?

review: Approve
Revision history for this message
Jelmer Vernooij (jelmer) wrote :

> LGTM 👍 Are these the same changes that are shipping in the lptools
> Debian/Ubuntu packages already?

Yeah, the package just includes these changes as a patch

Revision history for this message
Guruprasad (lgp171188) :
review: Needs Fixing
lp://staging/~jelmer/lptools/python3 updated
48. By Jelmer Vernooij

Remove debug command

Revision history for this message
Jelmer Vernooij (jelmer) wrote :

Fixed

Revision history for this message
Guruprasad (lgp171188) wrote :

Thank you for the valuable contribution!

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches