-
Notifications
You must be signed in to change notification settings - Fork 85
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Bitbutcket request ls shows more than just open PRs #193
Comments
what is the command to reproduce this from the cli? |
what is the command to reproduce this from the cli? For example, I do this and get some errors:
|
I have no idea on why creating PRs is broken. Also I was wrong about my initial statement to fix it the diff --git a/git_repo/services/ext/bitbucket.py b/git_repo/services/ext/bitbucket.py
index 5b2c43d..550b101 100644
--- a/git_repo/services/ext/bitbucket.py
+++ b/git_repo/services/ext/bitbucket.py
@@ -319,10 +319,11 @@ class BitbucketService(RepositoryService):
r.links['html']['href']
) for r in self.bb.repositoryPullRequestsInState(
owner=user,
- repository_name=repo,
- state='open'
+ repository_name=repo
) if not isinstance(r, dict) # if no PR is empty, result is a dict
)
+ yield "{}\t{:<60}\t{}"
+ yield 'id', 'title', 'URL'
for pull in sorted(requests):
try:
yield pull otherwise
|
Ok, but what I am typing is the expected way to create a repo? I can probably have a look at some point as long as I know the correct entry point. |
For some reason the bitbucket PR list shows all PRs, not just open ones.
Delete this line (and the comma on the previous line) and it works fine.
https://github.com/guyzmo/git-repo/blob/devel/git_repo/services/ext/bitbucket.py#L323
I did not look into why that fixes it.
The text was updated successfully, but these errors were encountered: