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 Declare::CLI SYNOPSIS to match what's designed in the source code #3

Merged
merged 1 commit into from Oct 7, 2015

Conversation

Projects
None yet
2 participants
@zhouzhen1
Contributor

zhouzhen1 commented Oct 6, 2015

This is for the CPAN PR Challenge.

Issues #1 and #2 are basically about inconsistency between the source and doc. The source calls a handler with ($cmd, $opts, @Args), while the doc suggests user to do ($opts, $argsref). See https://metacpan.org/source/EXODIST/Declare-CLI-0.008/lib/Declare/CLI.pm#L326

Now this PR matches the doc to the source, as well as a couple of minor fixes that prevents user from trying out the SYNOPSIS. But if you would prefer the behavior in the doc (no $cmd, and use $argsref instead of @Args), I can change it and fix the source.

exodist added a commit that referenced this pull request Oct 7, 2015

Merge pull request #3 from zhouzhen1/patch
fix Declare::CLI SYNOPSIS to match what's designed in the source code

@exodist exodist merged commit 1b045f0 into exodist:master Oct 7, 2015

@exodist

This comment has been minimized.

Owner

exodist commented Oct 7, 2015

released

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