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.

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

Support GitHub Enterprise #127

Open
wants to merge 10 commits into
base: master
Choose a base branch
from

Conversation

hirose31
Copy link

Thank you for greate product!

I want to use pep8speaks on my GitHub Enterprise. This PR enables that and fix some tweaks.

Could you possibly review?

to prevent aborting at ghrequest.pr_desc.lower() in helpers.comment_permission_check
query witouout parameters, returns all repositories which github user has
privileges owner, collaborator, organization_member.
https://developer.github.com/v3/repos/#list-your-repositories
for github in busy or slow github enterprise.
@auto-comment
Copy link

auto-comment bot commented May 28, 2019

💖 Thank you so much for creating the pull request! 💖

Have a look at the test workflow. The branch of this pull request is going to be deployed on Heroku with payload URL https://pep8speaks-pr-xxx.herokuapp.com . The repository https://github.com/OrkoHunter/test-pep8speaks is configured to test review deployments like this.

@OrkoHunter Create a review app for this PR on Heroku and update the Webhook URL of the test-pep8speaks app. Finally restart Travis to test this PR.
Remember to deploy the production app and revert the Webhook URL of the test GitHub app when the Pull Request is merged.

@OrkoHunter OrkoHunter temporarily deployed to pep8speaks-pr-127 May 31, 2019 08:15 Inactive
@OrkoHunter OrkoHunter temporarily deployed to pep8speaks-pr-127 May 31, 2019 08:17 Inactive
Sign up for free to 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.

None yet

2 participants