gomesfernanda/some-github-metrics

409 error in export_community_engagement when there is an empty repository

AlejandroReBa opened this issue · 3 comments

In this line:

for commits in repo.get_commits():

Next error is thrown when it finds an empty repository. (Catched by main try/catch(except))

409 {"message": "Git Repository is empty.", "documentation_url": "https://docs.github.com/enterprise/2.22/rest/reference/repos#list-commits"}

Can be solved with a nested try/catch to follow up with the next repository.

Moreover,

for commits in repo.get_commits():
    countcommit += 1

could be changed by

countcommit = repo.get_commits().totalCount

Implemented in this PR: PyGithub/PyGithub#820

Thanks

Hi Alejandro! I'm sorry for the delay. I just fixed this bug on the PR #16
Let me know if it works. Thanks!

I tried using repo.get_commits().totalCount but it was still throwing me an error, so I used a basic try and except to check if the repo is empty. Not the BEST solution haha but it works

@gomesfernanda thanks a lot for having a look at the issue 👍🏽

I just passed over it with some quick fixes for a self-demo before leaving the above message; I only wanted to notify you about that because your code was very handy to me.

Regards!