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

Update license information and information in Build.PL #2

Open
wants to merge 7 commits into
base: master
from

Conversation

Projects
None yet
2 participants
@bugdebugger

bugdebugger commented Mar 22, 2015

This PR updates/includes the licensing information to better fit the Kwalitee checks and the Perl Foundation guidelines.

It also adds a few requirements to Build.PL and removes a stale project homepage URL

This pull-request is part of the 2015 CPAN PR Challenge

bugdebugger added some commits Mar 22, 2015

Update license text and put it under own section
License information update was inspired by a Kwalitee warning
and the Perl Foundation licensing guidelines
http://www.perlfoundation.org/cpan_licensing_guidelines
Update homepage URL
open-exodus.net seems to be offline since ~2013
Tentatively specify minimum perl version
The one current at date of first module release
Build.PL Outdated
requires => {
'Fennec' => '1.012',
'Exporter::Declare' => '0.107',
'Exporter::Declare::Magic' => '0.107',
'Devel::Declare::Parser' => '0.017',
'Devel::Declare::Parser::Fennec' => '0.004',
'perl' => '5.10.0',

This comment has been minimized.

@exodist
Build.PL Outdated
},
meta_merge => {
resources => {
repository => 'http://github.com/exodist/Fennec-Declare',
bugtracker => 'http://github.com/exodist/Fennec-Declare/issues',
homepage => 'http://open-exodus.net/projects/Fennec-Declare'
homepage => 'http://blogs.perl.org/users/chad_exodist_granum/'

This comment has been minimized.

@exodist

exodist Mar 23, 2015

Owner

link to the github page, not my blog.

@exodist

This comment has been minimized.

Owner

exodist commented Mar 23, 2015

added a couple notes of things that need to be corrected for this pull request. Thanks!

@bugdebugger

This comment has been minimized.

bugdebugger commented Mar 24, 2015

Corrected the mentioned things. Thanks for taking the time to review this PR.

@bugdebugger

This comment has been minimized.

bugdebugger commented May 24, 2015

Is there still something left to do for me on this PR to bring it into merge form?

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