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

add use warnings; to *.pm files #65

Closed
wants to merge 1 commit into
base: master
from

Conversation

Projects
None yet
2 participants
@mishin
Contributor

mishin commented Aug 1, 2015

according to
http://cpants.cpanauthors.org/dist/ZMQ-LibZMQ2
module does not have use warnings;
I think that it's wrong
also Neil Bowers #pr-challenge push me to think about improve this distribution

@lestrrat

This comment has been minimized.

Show comment
Hide comment
@lestrrat

lestrrat Aug 1, 2015

Collaborator

The diffs show entire files being changed (linebreaks? tabs vs spaces? I can't immediately tell), so I can't accept this as is. Please make sure the change only includes relevant lines.

Collaborator

lestrrat commented Aug 1, 2015

The diffs show entire files being changed (linebreaks? tabs vs spaces? I can't immediately tell), so I can't accept this as is. Please make sure the change only includes relevant lines.

@mishin

This comment has been minimized.

Show comment
Hide comment
@mishin

mishin Aug 2, 2015

Contributor

sorry, I change it

Contributor

mishin commented Aug 2, 2015

sorry, I change it

@mishin mishin closed this Aug 2, 2015

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