Skip to content

Use Pygithub object instead of url_request - #29

Merged
stefanv merged 82 commits into
machine-shop:masterfrom
ttran428:main_tools
Apr 18, 2018
Merged

Use Pygithub object instead of url_request#29
stefanv merged 82 commits into
machine-shop:masterfrom
ttran428:main_tools

Conversation

@ttran428

Copy link
Copy Markdown
Contributor

Changed the functions find_pr_info and find_issues_info to use the github object instead to increase api calls.

@stefanv

Copy link
Copy Markdown
Contributor

Teddy, is this good to go? I see conflicts, and also it looks like you committed your key.

@ttran428

Copy link
Copy Markdown
ContributorAuthor

I deleted my key(I forgot to commit that change last time I think) and I fixed the conflicts by merging the upstream into my branch.

@stefanv

Copy link
Copy Markdown
Contributor

You now have doc/docs.zip committed.

After removing, please ask @SeraYang1 to review and merge if she's happy.

@ttran428

Copy link
Copy Markdown
ContributorAuthor

sounds good. I will ask her.

@stefanv
stefanv merged commit f2617eb into machine-shop:masterApr 18, 2018
SeraYang1 added a commit to SeraYang1/oss_devkit that referenced this pull request Apr 19, 2018
* upstream/master:
General cleanup (machine-shop#38)
Use Pygithub object instead of url_request (machine-shop#29)
SeraYang1 added a commit to SeraYang1/oss_devkit that referenced this pull request Apr 19, 2018
* upstream/master:
General cleanup (machine-shop#38)
Use Pygithub object instead of url_request (machine-shop#29)
SeraYang1 added a commit to SeraYang1/oss_devkit that referenced this pull request Apr 19, 2018
* upstream/master:
General cleanup (machine-shop#38)
Use Pygithub object instead of url_request (machine-shop#29)
SeraYang1 added a commit to SeraYang1/oss_devkit that referenced this pull request Apr 19, 2018
* upstream/master:
General cleanup (machine-shop#38)
Use Pygithub object instead of url_request (machine-shop#29)
SeraYang1 added a commit to SeraYang1/oss_devkit that referenced this pull request Apr 19, 2018
* upstream/master:
General cleanup (machine-shop#38)
Use Pygithub object instead of url_request (machine-shop#29)
SeraYang1 added a commit to SeraYang1/oss_devkit that referenced this pull request Apr 20, 2018
* upstream/master:
General cleanup (machine-shop#38)
Use Pygithub object instead of url_request (machine-shop#29)
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@ttran428@stefanv@SeraYang1