Bug #92566 for Test-Pod-No404s: Add an option to skip URLs

Wed Jan 29 18:25:48 2014 cpan [...] zoffix.com - Ticket created

Subject:

Add an option to skip URLs

As the subject says, and/or: do not check URLs written as code examples. For example, this POD fails: http://pastebin.ca/2599645 The issue is the sample URL https://USERNAME:PASSWORD@api.github.com/authorizations I don't see a means to tell this module to not try to test that URL.... -- Cheers, ZZ [ https://metacpan.org/author/ZOFFIX ]

Sat Nov 01 22:42:10 2014 APOCAL [...] cpan.org - Correspondence added

Hello, I agree with you! However, I'm not sure how you would like to control which URLs to skip? A regex? A simple list of hosts? Some callback coderef? Please let me know :) On Wed Jan 29 18:25:48 2014, ZOFFIX wrote: Show quoted text

> As the subject says, and/or: do not check URLs written as code > examples. > > For example, this POD fails: http://pastebin.ca/2599645 > > The issue is the sample URL > https://USERNAME:PASSWORD@api.github.com/authorizations > > I don't see a means to tell this module to not try to test that > URL....

-- ~Apocalypse

Sat Nov 01 22:42:11 2014 The RT System itself - Status changed from 'new' to 'open'

Tue Apr 28 06:27:59 2015 dolmen [...] cpan.org - Correspondence added

Le 2014-01-30 00:25:48, ZOFFIX a écrit :
Show quoted text

> As the subject says, and/or: do not check URLs written as code
> examples.

Test::Pod::No404s should only check POD links L<>.
URI in verbatim blocks or in something else than L<> should not checked by default, or at least not reported as fatal errors.

--
Olivier Mengué - http://perlresume.org/DOLMEN - https://gratipay.com/dolmen/

Tue Apr 28 17:08:53 2015 cpan [...] zoffix.com - Correspondence added

On Sat Nov 01 22:42:10 2014, APOCAL wrote: Show quoted text

> how you would like to control?

Sorry to respond so late. Seems I've missed a notification from RT. But a simple list of URLs seems a straightforward solution. Chances are the user will want to skip only a couple of URLs they list in their POD, so listing them in, say, a =for POD block would be quite easy. And I do side with dolmen's comment above: only URLs within L<> blocks should be tested. In which case, option to skip URLs isn't needed, since it's unlikely someone would put an uncheckable URL inside an L<> block. My two cents. -- Cheers, ZZ [ https://metacpan.org/author/ZOFFIX ]

Sat Jul 18 16:47:23 2015 ether [...] cpan.org - Correspondence added

On 2015-04-28 14:08:53, ZOFFIX wrote: Show quoted text

> On Sat Nov 01 22:42:10 2014, APOCAL wrote:

> > how you would like to control?

> > Sorry to respond so late. Seems I've missed a notification from RT. > But a simple list of URLs seems a straightforward solution. Chances > are the user will want to skip only a couple of URLs they list in > their POD, so listing them in, say, a =for POD block would be quite > easy. > > And I do side with dolmen's comment above: only URLs within L<> blocks > should be tested. In which case, option to skip URLs isn't needed, > since it's unlikely someone would put an uncheckable URL inside an L<> > block.

I agree with all this. See also https://rt.cpan.org/Ticket/Display.html?id=105937 It might still be desired to check URLs in other formats, but it certainly shouldn't be the default, and it can be left unimplemented as "YAGNI" until someone asks for it :) The text extraction logic is already done and can be retrieved from git anyway.