Change the API for get_remote_repositories().
Review Request #5903 — Created May 29, 2014 and submitted
get_remote_repositories()
's API was using plan IDs as a way of
identifying users vs. organizations, which led to some redundancy.
For example, "private" and "public" for GitHub meant the same thing
(users). GitHub in particular was assuming certain filter types for
repositories, which would be handy to specify from a caller's point of
view.The function has now been changed to remove the plan parameter and to
instead have a dedicated "owner_type" parameter that specifies what
"owner" refers to. The value is dependent on the hosting service, and
HostingService subclasses are expected to be smart about the defaults.It also now takes a "filter_type" parameter that allows for filtering
repositories. Possible values are dependent on the hosting service as
well. For instance, GitHub's allowed values consist of what their API
natively allows for their "type=" parameter in repository queries.
Unit tests pass.
Tested with my upcoming web API change, testing user and org lookups
and various filters.
-
This is a review from Review Bot. Tool: Pyflakes Processed Files: reviewboard/hostingsvcs/github.py reviewboard/hostingsvcs/tests.py reviewboard/hostingsvcs/service.py Ignored Files:
- Change Summary:
-
Fixed some alignment issues.
- Commit:
-
59ed286b922f27395bb5588234976e8a44d0b88936034e5355b52b3e22309f346da9a2ecd4612380
-
This is a review from Review Bot. Tool: PEP8 Style Checker Processed Files: reviewboard/hostingsvcs/github.py reviewboard/hostingsvcs/tests.py reviewboard/hostingsvcs/service.py Ignored Files: