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

Fix synopsis in App::CLI::Command::Help (PRC) #2

Closed
wants to merge 1 commit into
from

Conversation

Projects
None yet
2 participants
@jjatria

jjatria commented Aug 2, 2017

Using the old synopsis would result in the error

Can't locate object method "SUPER" via package "MyApp::Help" at lib/MyApp/Help.pm line 7.

This patch fixes that so it can be used as is.

@paultcochrane

This comment has been minimized.

Show comment
Hide comment
@paultcochrane

paultcochrane Aug 8, 2017

Owner

How did you test that? I'm not the original author, have just recently started maintenance of the project, so would like to extend the test suite (and my understanding) where possible.

Owner

paultcochrane commented Aug 8, 2017

How did you test that? I'm not the original author, have just recently started maintenance of the project, so would like to extend the test suite (and my understanding) where possible.

@paultcochrane

This comment has been minimized.

Show comment
Hide comment
@paultcochrane

paultcochrane Aug 8, 2017

Owner

Merged in 3917ca6. Many thanks!

Owner

paultcochrane commented Aug 8, 2017

Merged in 3917ca6. Many thanks!

@jjatria

This comment has been minimized.

Show comment
Hide comment
@jjatria

jjatria Aug 8, 2017

jjatria commented Aug 8, 2017

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