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

Enable strict and warnings for Kwalite metrcis #102

Closed
wants to merge 1 commit into
base: master
from

Conversation

Projects
None yet
3 participants
@jluis
Contributor

jluis commented Jul 16, 2015

Signed-off-by: Jose Luis Perez Diez jluis@escomposlinux.org

Entable strict and warnings for Kwalite metrcis
Signed-off-by: Jose Luis Perez Diez <jluis@escomposlinux.org>
@abraxxa

This comment has been minimized.

Show comment
Hide comment
@abraxxa

abraxxa Jul 16, 2015

Collaborator

It seems you don't know that use Moose enables warnings and strict.

Collaborator

abraxxa commented Jul 16, 2015

It seems you don't know that use Moose enables warnings and strict.

@jluis

This comment has been minimized.

Show comment
Hide comment
@jluis

jluis Jul 16, 2015

Contributor

El Dijous 16 Juliol 2015, a les 05:52:35, Alexander Hartmaier va escriure:

It seems you don't know that use Moose enables warnings and strict.

I know but it don't get reflected in the kwalite metrics see:

http://cpants.cpanauthors.org/dist/HTML-FormHandler

Contributor

jluis commented Jul 16, 2015

El Dijous 16 Juliol 2015, a les 05:52:35, Alexander Hartmaier va escriure:

It seems you don't know that use Moose enables warnings and strict.

I know but it don't get reflected in the kwalite metrics see:

http://cpants.cpanauthors.org/dist/HTML-FormHandler

@abraxxa

This comment has been minimized.

Show comment
Hide comment
@abraxxa

abraxxa Jul 16, 2015

Collaborator

Then please submit a patch for cpants, working around it by adding thousands of lines to each module which uses one of the modules that enable strict and warnings is no good solution in my opinion.

Collaborator

abraxxa commented Jul 16, 2015

Then please submit a patch for cpants, working around it by adding thousands of lines to each module which uses one of the modules that enable strict and warnings is no good solution in my opinion.

@gshank

This comment has been minimized.

Show comment
Hide comment
@gshank

gshank Sep 13, 2015

Owner

As Abraxxa said, putting a bunch of unnecessary "use strict; use warnings;" into FormHandler is not the proper solution. The problem is with the Kwalitee metrics.

Owner

gshank commented Sep 13, 2015

As Abraxxa said, putting a bunch of unnecessary "use strict; use warnings;" into FormHandler is not the proper solution. The problem is with the Kwalitee metrics.

@gshank gshank closed this Sep 13, 2015

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