Merge ~vorlon/ubuntu-dev-tools:pm-helper into ubuntu-dev-tools:main

Proposed by Steve Langasek
Status: Needs review
Proposed branch: ~vorlon/ubuntu-dev-tools:pm-helper
Merge into: ubuntu-dev-tools:main
Diff against target: 324 lines (+278/-0)
5 files modified
debian/control (+2/-0)
doc/pm-helper.1 (+44/-0)
pm-helper (+149/-0)
requirements.txt (+1/-0)
ubuntutools/utils.py (+82/-0)
Reviewer Review Type Date Requested Status
Bryce Harrington Approve
Benjamin Drung Pending
Ubuntu Core Development Team Pending
Ubuntu Development Team Pending
Review via email: mp+444677@code.staging.launchpad.net

Description of the change

As part of my +1 maintenance shift this week I scratched the itch of wanting a commandline tool to streamline identifying a package to work on in update_excuses and taking a lock on it. Initially this just takes a package name and manages update-excuse bugs for it but eventually it should be extended to scroll through the list of packages in update_excuses (oldest first) and pick one, then grab information from various sources (launchpad, autopkgtest.u.c, Debian archive, buildd.debian.org, ci.debian.net) to make suggestions on fixing.

Very raw at the moment - no tests yet.

To post a comment you must log in.
Revision history for this message
Bryce Harrington (bryce) wrote :

Cool, hopefully my suggestions inline are useful. I look forward to how this develops.

review: Needs Fixing
Revision history for this message
Steve Langasek (vorlon) wrote :

Thanks for the review, Bryce! I've implemented most of your suggestions, and responded to comments. Would you care to re-review?

Revision history for this message
Bryce Harrington (bryce) wrote :

Looks great Steve, the updates all look great. This'll make a solid start for the tool. The man page is a nice touch, I'm often too lazy about making those. :-)

review: Approve
Revision history for this message
Benjamin Drung (bdrung) wrote :

This script has zero test cases which makes maintaining it hard. I would expect new code to come with test cases.

There was an error fetching revisions from git servers. Please try again in a few minutes. If the problem persists, contact Launchpad support.

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