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

add travis to test the build CPR #7

Closed
wants to merge 1 commit into
base: master
from

Conversation

Projects
None yet
2 participants
@jluis

jluis commented Apr 24, 2017

I was assigned this distribution on the April CPAN pull request challenge.

I created the travis for myself to be able to work systems where I don't have a proper developer environment but I can use Github to edit the files.

I'm plan to put at least a README on master stating the obvious that uses Dist::Zilla to build the distro and explain the extructure of the repository.

What do You think?

@mschout

This comment has been minimized.

Show comment
Hide comment
@mschout

mschout Apr 24, 2017

Owner

Actually the easiest way to do this is to just flip the "use_travis" option on under [@mschout] in dist.ini. I have just done this and pushed the .travis.yml that results from a "dzil build" with this option turned on.

A README.md for github would be nice. If you go that route add it to MANIFEST.SKIP so that it does not get bundled by dist zilla.

Owner

mschout commented Apr 24, 2017

Actually the easiest way to do this is to just flip the "use_travis" option on under [@mschout] in dist.ini. I have just done this and pushed the .travis.yml that results from a "dzil build" with this option turned on.

A README.md for github would be nice. If you go that route add it to MANIFEST.SKIP so that it does not get bundled by dist zilla.

@jluis

This comment has been minimized.

Show comment
Hide comment
@jluis

jluis Apr 24, 2017

Thanks for your fast reply I will Just do The README on another PR and try to learn more on the build process of this distributiion

jluis commented Apr 24, 2017

Thanks for your fast reply I will Just do The README on another PR and try to learn more on the build process of this distributiion

@mschout

This comment has been minimized.

Show comment
Hide comment
@mschout

mschout Apr 24, 2017

Owner

FWIW, its the same as pretty much every other Dist::Zilla packaged module:

cpanm Dist::Zilla
dzil authordeps --missing | cpanm
dzil build
Owner

mschout commented Apr 24, 2017

FWIW, its the same as pretty much every other Dist::Zilla packaged module:

cpanm Dist::Zilla
dzil authordeps --missing | cpanm
dzil build
@mschout

This comment has been minimized.

Show comment
Hide comment
@mschout

mschout Jul 31, 2017

Owner

travis is enabled, so closing this PR

Owner

mschout commented Jul 31, 2017

travis is enabled, so closing this PR

@mschout mschout closed this Jul 31, 2017

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