Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Consider using a github lib to access the API #64

Open
tyll opened this issue Jan 9, 2020 · 4 comments
Open

Consider using a github lib to access the API #64

tyll opened this issue Jan 9, 2020 · 4 comments
Assignees
@xjezda00 @ukulekek

Comments

@tyll
Copy link
Member

tyll commented Jan 9, 2020

GitHub Python libs might already solve some problems that we are seeing with the API (pagination might be needed, handling request limits). On
Fedora there are these packages: python3-pygithub.noarch and python3-github3py.noarch Maybe they already solve such issues so that we do not need to re-invent the wheel.
@yontalcar
Copy link
Contributor

yontalcar commented Jan 9, 2020 •
edited
Loading

Of these two, PyGithub looks better.

@richm
Copy link
Contributor

richm commented Jul 1, 2020

Of these two, PyGithub looks better.

I just ran into an issue where I needed pagination - I'm bumping the priority of this one - @nhosoi

@richm
Copy link
Contributor

richm commented Jul 1, 2020

PyGithub does support paginated get for most objects/types

@pcahyna
Copy link
Member

pcahyna commented Jul 1, 2020

@ukulekek I believe you have been considering reusing some common code to set GitHub statuses as well - could it be used here?

@pcahyna pcahyna assigned ukulekek and unassigned yontalcar Jul 1, 2020
@nhosoi nhosoi mentioned this issue Jul 20, 2020
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@xjezda00 xjezda00

@ukulekek ukulekek

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
6 participants
@richm @tyll @xjezda00 @yontalcar @ukulekek @pcahyna