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

Remove all Github update checks #761

Merged
merged 1 commit into from
Oct 7, 2016

Conversation

zachriggle
Copy link
Member

These were nice-to-have, but as long as PyPi is updated and functioning
properly, we don't need to check Github.

Fixes #756

These were nice-to-have, but as long as PyPi is updated and functioning
properly, we don't need to check Github.

Fixes Gallopsled#756
@zachriggle
Copy link
Member Author

Merging despite coverage decrease, since there's less code now and no still-relevant tests were removed.

No need for this to be backported to beta, since the code still works, just has CI issues when it gets hit frequently from Travis.

@zachriggle zachriggle merged commit aec3fa6 into Gallopsled:dev Oct 7, 2016
TethysSvensson pushed a commit that referenced this pull request Oct 18, 2016
These were nice-to-have, but as long as PyPi is updated and functioning
properly, we don't need to check Github.

Fixes #756
@TethysSvensson TethysSvensson added this to the 3.2.0 milestone Oct 18, 2016
@zachriggle zachriggle deleted the github-updates branch January 4, 2017 18:03
Kyle-Kyle pushed a commit to Kyle-Kyle/pwntools that referenced this pull request Apr 25, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants