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

Test units, docs, less bloat, and travis #2

Merged
merged 1 commit into from Jul 5, 2017

Conversation

Projects
None yet
3 participants
@ghost

ghost commented Jul 6, 2016

Build Status

  • Become an AfterBuild plugin instead of a InstallTool plugin

  • Don't use Moose::Autobox in an effort to keep lean

  • Only search within the root of the dist and only look for
    files named change, changes, or changelog (case insenstive)

  • Added test suites and corpus for test suites

  • Log messages vs printing to STDOUT

  • Better error messages

  • Revise description

  • Add documentation about file name conventions

  • Tidy up

  • Add .travis.yml CI for use with github

  • Add README.md for github

    Hi @fayland... This is ready for release. Doing this as a PR for documentation purposes and it is a big change set so wanted to also give you a chance to chime in. I don't believe I have write-access so I can not merge this myself.

Note that .travis.yml has been added to the repo, I recommend setting up a travis account if you don't already have one. The build image in README.md currently points to an account which may or may not exist :)

Test units, docs, less bloat, and travis
  - Become an AfterBuild plugin instead of a InstallTool plugin
  - Don't use Moose::Autobox in an effort to keep lean
  - Only search within the root of the dist and only look for
    files named change, changes, or changelog (case insenstive)
  - Added test suites and corpus for test suites
  - Log messages vs printing to STDOUT
  - Better error messages
  - Revise description
  - Add documentation about file name conventions
  - Tidy up
  - Add .travis.yml CI for use with github
  - Add README.md for github
@stphnlyd

This comment has been minimized.

Show comment
Hide comment
@stphnlyd

stphnlyd Jul 5, 2017

I agree that it's not necessary to depend on Moose::Autobox as that thing brings in a set of dependencies. On my Solaris host here somehow autobox just does not build for Perl 5.26 now...

stphnlyd commented Jul 5, 2017

I agree that it's not necessary to depend on Moose::Autobox as that thing brings in a set of dependencies. On my Solaris host here somehow autobox just does not build for Perl 5.26 now...

@fayland fayland merged commit 1321561 into fayland:master Jul 5, 2017

@fayland

This comment has been minimized.

Show comment
Hide comment
@fayland

fayland Jul 5, 2017

Owner

I'm very SORRY that I missed it. Sorry.

Owner

fayland commented Jul 5, 2017

I'm very SORRY that I missed it. Sorry.

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