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

Added a contributing section to the readme #8

Merged
merged 1 commit into from Mar 14, 2015

Conversation

Projects
None yet
2 participants
@mrcaron
Contributor

mrcaron commented Mar 13, 2015

Just a few added notes on contributing to this. I have setup a really nice Vagrant environment that allowed me to develop on my windows box fairly easily (because Dist::Zilla wants to explode on Windows). I thought it was useful for others.

@nigelm nigelm merged commit 57474cc into nigelm:master Mar 14, 2015

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
@nigelm

This comment has been minimized.

Owner

nigelm commented Mar 14, 2015

Hi,

I merged that, and then realised it was into README.pod which is autogenerated by Dist::Zilla!
So I have moved the text into the pm file where it should have been.
I have also merged the other open tickets, and then applied a set of perltidy settings across everything - my sincere apologies if this manages to screw things up for you, although a rebase should clear it.

There is a trial release pushed out now to let the cpantesters builders and anyone with some interest check things out.

@mrcaron

This comment has been minimized.

Contributor

mrcaron commented Mar 16, 2015

Awesome! Glad I could help.
On Sat, Mar 14, 2015 at 1:29 PM Nigel Metheringham notifications@github.com
wrote:

Hi,

I merged that, and then realised it was into README.pod which is
autogenerated by Dist::Zilla!
So I have moved the text into the pm file where it should have been.
I have also merged the other open tickets, and then applied a set of
perltidy settings across everything - my sincere apologies if this manages
to screw things up for you, although a rebase should clear it.

There is a trial release pushed out now to let the cpantesters builders
and anyone with some interest check things out.


Reply to this email directly or view it on GitHub
#8 (comment).

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