-
Notifications
You must be signed in to change notification settings - Fork 61
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
CGI::Alternatives dep #186
Comments
Another good idea. Feel free to make a PR, otherwise i'll get around to looking at this soon enough. |
You know, i'm having second thoughts on this - my experience with vendor package managers and the like is that they will pull out any dependencies that aren't actually used by a distribution. So i think it would be better to have a link rather than pulling it in as a dep. |
No prob, a hyperlink works. CGI::Alternatives is more discursive rather than technical documentation anyway, so it doesn't really need to be installed as a manual page. |
I'm happy with what we've got here - the README.md links to the metacpan dist, whereas the POD has the L<> tags that will play well with CPAN/metacpan and the man pages. |
Since the documentation for CGI mentions CGI::Alternatives, it would be a good idea to add a package dependency on that so that the alternatives documentation is installed. (Either that, or link to it as a URI rather than with L<> links to 'manual pages'.)
The text was updated successfully, but these errors were encountered: