Fix .travis.yml so that it is valid YAML travis can parse #19

Merged
merged 1 commit into from Mar 14, 2012

2 participants

@michaelklishin

Please use lint.travis-ci.org (see this doc guide).

@michaelklishin

To clarify: there were tabs in the file. YAML requires you to use (commonly two) spaces for indentation in arrays.

@melo melo merged commit 276f6ca into PerlRedis:master Mar 14, 2012
@melo
Home of Redis.pm and related modules member

Thanks @michaelklishin

I really hate white-space-based formats :(

@michaelklishin

Now you have build matrix with Perls but cpanm --quiet --installdeps --notest . exits with non-0. You can override install: to not use --quiet:

install: cpanm --installdeps --notest .

or do any other dependency installation routine your project needs.

@melo
Home of Redis.pm and related modules member

I'll try that, but the problem is probably related to the fact that I'm using Dist::Zilla, and not the standard setup for a Perl module.

When I run dzil release on my laptop, Dist::Zilla creates all the boilerplate that Perl modules require, and then it packs that and submits the final tarball to PAUSE.

But that boilerplate is not on my repo, given that its easly generated. And cpanm --installdeps expects it to be there.

I believe I can have a Dist::Zilla plugin push to git the final build result, with the full boilerplate. If I can convince travis-ci to watch a particular branch, my problems will be solved. I'll need to study this a bit more.

Thank you for your time debugging this. I will try your suggestion to override install to make sure I'm on the right track though.

@michaelklishin

There is a project using Dist::Zilla and we will probably preinstall it soon.

@michaelklishin

travis-ci will build all branches github fires hooks for. This also means that .travis.yml needs to be on all active branches.

@melo
Home of Redis.pm and related modules member

Looking at the project you mentioned, and testing it now. Looks simple enough. I'll probably include it on my default setup for all modules.

@melo
Home of Redis.pm and related modules member

I think fec78a4 will work the best. Waiting for the results of the build to check.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment