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

[GitHub] Fix Travis-CI GitHub rate limiting #1094

Closed
wants to merge 2 commits into from

Conversation

RedSparr0w
Copy link
Member

Attempt to fix #979 limit on GitHub request.
Travis-CI Environment variable github_token will need to be set for this to work though.

Travis-CI Enviroment variable `github_token` will need to be set.
@paulmelnikow paulmelnikow added the developer-experience Dev tooling, test framework, and CI label Sep 25, 2017
@paulmelnikow
Copy link
Member

Thanks for this. I'd rather not add one-off logic to deal with our internal uses. While it'd be nice to provide a way to load secrets from the environment (rather than the filesystem) I'd want to do that in a holistic way. I opened #1095 which is an alternate approach. See what you think.

unassigned githubUserTokens -> process.env.github_token
@RedSparr0w RedSparr0w closed this Sep 26, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
developer-experience Dev tooling, test framework, and CI
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Github rate limits cause transient service test failures in CI
2 participants