Skip to content
This repository has been archived by the owner on Oct 15, 2022. It is now read-only.

.travis.yml: Remove sudo: false #1456

Merged
merged 1 commit into from
Jan 29, 2015
Merged

.travis.yml: Remove sudo: false #1456

merged 1 commit into from
Jan 29, 2015

Conversation

jagtalon
Copy link
Member

Since .travis.yml is autogenerated by https://github.com/SineSwiper/Dist-Zilla-TravisCI, we can't just modify that file ourselves. We'd have to wait for SineSwiper/Dist-Zilla-TravisCI#25 to get fixed first.

@moollaza

Since .travis.yml is autogenerated by https://github.com/SineSwiper/Dist-Zilla-TravisCI, we can't just modify that file ourselves. We'd have to wait for SineSwiper/Dist-Zilla-TravisCI#25 to get fixed first.
@moollaza
Copy link
Member

Understood. Thanks @jagtalon

@moollaza moollaza closed this Jan 29, 2015
@moollaza moollaza reopened this Jan 29, 2015
moollaza added a commit that referenced this pull request Jan 29, 2015
.travis.yml: Remove sudo: false
@moollaza moollaza merged commit 0d5c754 into master Jan 29, 2015
@moollaza moollaza deleted the jag/travis branch January 29, 2015 20:49
@jagtalon
Copy link
Member Author

@moollaza Sorry I completely linked to the wrong repo! We still can't do it, but just wanna point out that we're using https://github.com/Getty/p5-dist-zilla-plugin-travisci

@moollaza
Copy link
Member

@jagtalon wow, good catch I totally thought we were using the other one. We should definitely switch to that one (no one is maintaining ours AFAIK & the other looks like it's well supported/reviewed). Hopefully then we won't need to have the actual file in the repo, which is very confusing!

@kentfredric
Copy link

Hopefully then we won't need to have the actual file in the repo, which is very confusing!

There's no way to have the file missing without having a build branch.

And that will mean pull requests don't get tested.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants