Skip to content
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

Ensure expected dist name is in metadata #1

Merged
merged 4 commits into from
Dec 4, 2014

Conversation

neilb
Copy link
Contributor

@neilb neilb commented Dec 3, 2014

Hi George,

This dist has the wrong distname in the metadata file, so I corrected that.

I made some other minor changes:

  • reformatted the Changes file to follow convention
  • added "use 5.010" to the module (which matches what is specified in the metadata)
  • added mention of the github repo to the doc

Cheers,
Neil

@teodesian
Copy link
Owner

Thanks, stuff I didn't know. Will work on the rest of my modules for compliance.

teodesian added a commit that referenced this pull request Dec 4, 2014
Ensure expected dist name is in metadata, update changes file to match CPAN spec, declare perl version in module, and add REPOSITORY section to POD
@teodesian teodesian merged commit f7b5127 into teodesian:master Dec 4, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants