-
Notifications
You must be signed in to change notification settings - Fork 813
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
Fix branch builds for this repo #500
Comments
@yuvipanda Do you think the latest addition of chartpress and the changes to |
We currently end up with two builds on travis running, this is not the correct behavior i figure. |
This is an example of an troublesome behavior, where we get two builds, one because we get a PR, and one because we added a branch. Actionplan:
|
Why not recommend against having private branches in this repository? We operate via PRs for all change so people have their own fork already and can do what they like there. My preference for this is part "keeping the master repository tidy" and "only special people can make branches here and I don't like having special people". |
@betatim absolutely agree :) |
-> close this issue? Or is there a GitHub setting we can/should use to enforce this? |
I don't think that GitHub offers a straightforward way to prevent new branch creation. I'm going to close. |
Currently, if you push to a branch in this repo, Travis fails.
We should fix this!
The text was updated successfully, but these errors were encountered: