Skip to content
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

perldoc $module rather than perldoc $main_module (RT#101917) #7

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

pwr22
Copy link

@pwr22 pwr22 commented Feb 4, 2015

Uses a Pod::Weaver function to pull out the correct module name per file being processed

@pwr22 pwr22 force-pushed the perldoc-by-module branch from 028a126 to 510d702 Compare February 4, 2015 23:36
@kentfredric
Copy link
Contributor

If you want, this PR can now be implemented in terms of PPIx::DocumentName which is specially designed to cover this usecase without needing to either re-implement it or without having to call private methods =)

https://metacpan.org/pod/release/KENTNL/PPIx-DocumentName-0.001002/lib/PPIx/DocumentName.pm#USAGE

@pwr22 pwr22 force-pushed the perldoc-by-module branch 2 times, most recently from 6d8857f to 1bbefc5 Compare February 10, 2015 18:45
@pwr22
Copy link
Author

pwr22 commented Feb 10, 2015

@kentfredric It's done, thanks for making that module :)

@pwr22
Copy link
Author

pwr22 commented Oct 30, 2015

@apocalypse Any chance that this could be merged?

@shlomif
Copy link

shlomif commented Oct 18, 2017

Hi! Thanks for this PR, it was merged to my fork. I am the new comaint of this dist - https://github.com/shlomif/perl-pod-weaver-section-support

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants