Merge ppa-dev-tools:fix-lp1990244-create-for-owner into ppa-dev-tools:main

Proposed by Bryce Harrington
Status: Merged
Merged at revision: 9dbaa6f713ae958ad415607c771549434fa1db93
Proposed branch: ppa-dev-tools:fix-lp1990244-create-for-owner
Merge into: ppa-dev-tools:main
Diff against target: 732 lines (+207/-111)
8 files modified
NEWS.md (+9/-0)
ppa/ppa.py (+35/-29)
ppa/ppa_group.py (+9/-9)
scripts/ppa (+38/-18)
tests/helpers.py (+3/-2)
tests/test_ppa.py (+32/-32)
tests/test_ppa_group.py (+5/-5)
tests/test_scripts_ppa.py (+76/-16)
Reviewer Review Type Date Requested Status
Lena Voytek (community) Approve
Canonical Server packageset reviewers Pending
PpaDevTools Developers Pending
Canonical Server Reporter Pending
Review via email: mp+443452@code.staging.launchpad.net

Description of the change

This adds the ability to create PPAs against other teams that your LP user account has access to, fixing LP: #1990244. You can specify the team by a command line argument, config file variable, or the ppa address.
For example:

$ ./scripts/ppa create --team ppa-dev-tools-devs foobar
PPA 'foobar' created for the following architectures:

  i386, amd64, armhf, ppc64el, s390x, arm64, powerpc

The PPA can be viewed at:

  https://launchpad.net/~ppa-dev-tools-devs/+archive/ubuntu/foobar

...

$ ./scripts/ppa create ppa-dev-tools-devs/example
PPA 'example' created for the following architectures:

  i386, amd64, armhf, ppc64el, s390x, arm64, powerpc

The PPA can be viewed at:

  https://launchpad.net/~ppa-dev-tools-devs/+archive/ubuntu/example

...

To post a comment you must log in.
Revision history for this message
Christian Ehrhardt  (paelzer) wrote :

I like this, I do not feel I was able to do a full review - but for whatever it is worth it LGTM.
Furthermore I have left a few small comments inline for you to think about.

Revision history for this message
Lena Voytek (lvoytek) wrote :

LGTM, added a few comments below for formatting/descriptions

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

Thanks, I've updated with the changes as suggested and landed:

Total 0 (delta 0), reused 0 (delta 0), pack-reused 0
To git+ssh://git.launchpad.net/ppa-dev-tools
   2a8e2ef..9dbaa6f main -> main

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

to all changes: