Skip to content
This repository has been archived by the owner on Jan 25, 2019. It is now read-only.

Make travis use correct branch, not always master #6

Closed
wants to merge 1 commit into from

Conversation

sideshowbarker
Copy link
Contributor

Without this change, travis fails with error: pathspec 'master' did not match any file(s) known to git.

@tagawa
Copy link
Contributor

tagawa commented Jan 5, 2015

My git fu is not strong enough to merge this - any other takers?

@sideshowbarker
Copy link
Contributor Author

My git fu is not strong enough to merge this - any other takers?

@jgraham wrote that rakefile and the travis setup, so he probably ought to doublecheck this to make sure it's not going to break anything.

@jgraham
Copy link
Member

jgraham commented Jan 5, 2015

I don't understand why this is needed. Running travis on !master seems bad because it will try to push that other branch into production.

@sideshowbarker
Copy link
Contributor Author

I don't understand why this is needed. Running travis on !master seems bad because it will try to push that other branch into production.

OK, I didn't know that the travis setup was actually always pushing changes to production.

Given that, I've now disabled the "Build pull requests" option at https://travis-ci.org/w3c/testtwf-website-src/settings

We want that disabled, right?

@jgraham
Copy link
Member

jgraham commented Jan 5, 2015

Yeah, I thought that was already disabled, but that might have been my test setup.

@jgraham jgraham closed this Jan 5, 2015
@sideshowbarker sideshowbarker deleted the sideshowbarker/fix-travis branch June 22, 2015 23:51
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants