[WIP] HostingServices: Bitbucket get_remote_repositories, get_commits, get_branches
Review Request #5450 — Created Feb. 10, 2014 and discarded
Added get_remote_repositories, get_commits, get_branches for Bitbucket
Command line testing done, unit tests will be added after tests in request #5432 are approved.
Description | From | Last Updated |
---|---|---|
Same as my other review concern - can we guarantee that this will exit without putting our full trust in … |
mike_conley | |
No space before the """. |
chipx86 | |
Can be one statement. |
chipx86 | |
Blank line before this. |
chipx86 | |
Blank line before this. |
chipx86 | |
BLANK LINE BEFORE THIS. |
chipx86 | |
One variable setting per line. Same below. |
chipx86 | |
Blank line between these. |
chipx86 | |
next is a reserved word. |
chipx86 | |
We don't use the if .. else .. form in our code. Should just be a standard if statement. |
chipx86 | |
The """ should go on the previous line. |
chipx86 | |
Blank line before this. |
chipx86 | |
Blank line before this. |
chipx86 | |
No u prefixes. |
chipx86 | |
And this. |
chipx86 | |
A quicker and nicer way of doing this is: results = [ Branch(...) for name, info in six.iteritems(branches) ] You … |
chipx86 | |
You can just do: if not results: |
chipx86 | |
And this. |
chipx86 | |
This line is too long. Maybe shorten the summary and go into more detail in the description. |
chipx86 | |
Blank line above. |
chipx86 | |
Blank line before this. |
chipx86 | |
Did you mean to capitalize all of "THE" ? |
chipx86 | |
And this. |
chipx86 | |
Blank line before this. |
chipx86 |
- Change Summary:
-
Added get_commits, get_branches
- Description:
-
~ Added get_remote_repositories for Bitbucket
~ Added get_remote_repositories, get_commits, get_branches for Bitbucket
- Groups:
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
A quicker and nicer way of doing this is:
results = [ Branch(...) for name, info in six.iteritems(branches) ]
You then won't need to set
results = []
above.Also note the
six.iteritems
. We use a module calledsix
to provide compatibility with Python 3. One of the things we must use it for is anyiter*
function.If you do any iteration anywhere else in your code (I may have missed these), please update them to use
six
. -
-
-
-
-
-
-
-