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.

Sign up for GitHub

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

Jump to bottom

Travis CI not running on PRs from local branches #337

Closed
martinb3 opened this issue Jul 8, 2015 · 3 comments
Closed

Travis CI not running on PRs from local branches #337

martinb3 opened this issue Jul 8, 2015 · 3 comments
Assignees
@karmi
Labels
Bug Something isn't working
Milestone
1.0.0 release to ...

Comments

@martinb3
Copy link
Contributor

martinb3 commented Jul 8, 2015

See the pull request page from TravisCI. It doesn't show, for example, #336. The PR wasn't built at all, as far as I can tell.
@martinb3 martinb3 added the Bug Something isn't working label Jul 8, 2015
@karmi
Copy link
Contributor

karmi commented Jul 9, 2015

This is weird... Travis seems to be configured properly:

cookbook-travis

@davidski
Copy link

davidski commented Jul 9, 2015

Does the limitation of Travis builds to the next branch via these (https://github.com/elastic/cookbook-elasticsearch/blob/master/.travis.yml#L3-L5) lines in the .travis.yml also turn off PR builds?

@martinb3
Copy link
Contributor Author

martinb3 commented Jul 9, 2015

That seems likely.

@martinb3 martinb3 added this to the 1.0.0 release to Supermarket milestone Jul 16, 2015
martinb3 added a commit that referenced this issue Jul 16, 2015
@martinb3
Run CI on master branch again, after rename
75f6e7e 
Run TravisCI on master branch again, since we renamed next branch to master a bit ago.

Fixes #337.
@martinb3 martinb3 mentioned this issue Jul 16, 2015
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@karmi karmi

Labels
Bug Something isn't working
Projects
None yet
Milestone
1.0.0 release to Supermarket
Development

No branches or pull requests
3 participants
@karmi @martinb3 @davidski