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

Applied reporter supplied patch for RT #32954. #7

Closed
wants to merge 1 commit into
base: master
from

Conversation

Projects
None yet
2 participants
@estrabd

estrabd commented Feb 15, 2016

Original issue,

https://rt.cpan.org/Ticket/Display.html?id=32954

Applied this patch and all tests still ran successfully.

@neilb

This comment has been minimized.

Show comment
Hide comment
@neilb

neilb Feb 15, 2016

Owner

I'm skimming quickly, but is that usage of Data::Dumper left over from development testing?

Owner

neilb commented Feb 15, 2016

I'm skimming quickly, but is that usage of Data::Dumper left over from development testing?

@estrabd

This comment has been minimized.

Show comment
Hide comment
@estrabd

estrabd Feb 15, 2016

Ah, part of the patch I didn't see. I'll clean it up and will resubmit.

estrabd commented Feb 15, 2016

Ah, part of the patch I didn't see. I'll clean it up and will resubmit.

@estrabd estrabd closed this Feb 17, 2016

@estrabd

This comment has been minimized.

Show comment
Hide comment
@estrabd

estrabd Feb 17, 2016

Please see PR #9, thank you.

estrabd commented Feb 17, 2016

Please see PR #9, thank you.

Brett Estrade
Applied reporter supplied patch for RT #32954.
Original issue,

https://rt.cpan.org/Ticket/Display.html?id=32954

Applied this patch and all tests still ran successfully.

Modified original patch to remove "use Data::Dumper" from
t/getopt.t
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment