Skip to content

User repositories list -l failing on github if one of repositories empty #83

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

Closed
pyhedgehog opened this issue Dec 21, 2016 · 1 comment · Fixed by #86
Closed

User repositories list -l failing on github if one of repositories empty #83

pyhedgehog opened this issue Dec 21, 2016 · 1 comment · Fixed by #86
Assignees
Labels

Comments

@pyhedgehog
Copy link
Contributor

$ git hub ls -l pyhedgehog
Status  Commits Reqs    Issues  Forks   Coders  Watch   Likes   Lang    Modif           Name
F       6251    0       0       0       141     0       0       JavaScript      Dec 14 2014     pyhedgehog/addon-sdk
F       132     0       0       0       2       0       0       ?       Nov 19 2015     pyhedgehog/api-guide
Fatal error: 409 Git Repository is empty.

It's true - pyhedgehog/choco-packages is empty, but it's not reason for failing.

@guyzmo
Copy link
Owner

guyzmo commented Dec 21, 2016

yup 👍 thank you

also, list -l is failing because it's hammering the API. I'll need to add some throttling to the function.

@guyzmo guyzmo added the bug label Dec 21, 2016
@guyzmo guyzmo self-assigned this Dec 21, 2016
@guyzmo guyzmo added the backlog label Dec 21, 2016
guyzmo added a commit that referenced this issue Dec 23, 2016
fixes #83

Signed-off-by: Guyzmo <guyzmo+github@m0g.net>
guyzmo added a commit that referenced this issue Dec 28, 2016
🚧 Features

* Support for insecure connection through config option (fixes #66)
* Made gitlab feature complete:
 * Added support for list command in gitlab (cf #78)
 * Added support for gitlab gists (fixes #12)
 * Added support for gitlab merge requests (fixes #10)

🚒 Bugfixes

* Bad exceptions handling mistakes (fixes #84, fixes #85)
* Missing edge case handling in list command (fixes #83)

💄 Cosmetics

* Updated README with new config option
* Added @rnestler to contributors
* Updated TODO list

Signed-off-by: Guyzmo <guyzmo+github+pub@m0g.net>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants