You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Pull requests don't get loaded with an API token, so all tests fail against PRs (see #9).
Current solution is to create a local branch for the pull request which then tests run against, but it's a bit frustrating for contributors not to be able to test the PR.
@Aravind741 could you please try adding your own mapbox API keys to your fork as an environmental variable through the travis settings (rather than the .travis.yml)?
Settings URL should be something like https://travis-ci.org/aravind741/go-mapbox/settings
If that sorts out the PR test problem I will add it to the readme.
The text was updated successfully, but these errors were encountered:
Which updates the push build in the github PR view, but not the PR build.
Then you have to admin override the PR (with one test passing) to merge it :-/
Pull requests don't get loaded with an API token, so all tests fail against PRs (see #9).
Current solution is to create a local branch for the pull request which then tests run against, but it's a bit frustrating for contributors not to be able to test the PR.
@Aravind741 could you please try adding your own mapbox API keys to your fork as an environmental variable through the travis settings (rather than the .travis.yml)?
Settings URL should be something like https://travis-ci.org/aravind741/go-mapbox/settings
If that sorts out the PR test problem I will add it to the readme.
The text was updated successfully, but these errors were encountered: