Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with HTTPS or Subversion.

Download ZIP

Loading…

Don't index commonly not-indexed directories (t xt inc) #390

Closed
rwstauner opened this Issue · 6 comments

2 participants

@rwstauner
Owner

Besides the meta no_index
PAUSE skips anything in t/ and inc/:
https://github.com/andk/pause/blob/master/lib/PAUSE/dist.pm#L648

I think xt/ would be something to skip as well.

This test file is listed as documentation for the string of pod contained inside it:
https://metacpan.org/source/MARCEL/Pod-Markdown-1.110730/t/misc.t

@monken
Owner

Hey there,

https://api.metacpan.org/file/MARCEL/Pod-Markdown-1.110730/t/misc.t has indexed: false, the indexer already sets the indexed: false property on files in t/ xt/ inc/ blib/ etc.
What are you referring to exactly?

@rwstauner
Owner

Sorry, guess i forgot to mention the important part (what i was looking at on metacpan)...
now i can't remember :-/

On this page it says "documentation" for that file: https://metacpan.org/source/MARCEL/Pod-Markdown-1.110730/t

The "documentation" link on the release page points to the bin/ script though, so that part seems ok:
https://metacpan.org/release/MARCEL/Pod-Markdown-1.110730/

Maybe the "documentation" thing in the "browse" view confused me....
maybe it was late...

@monken
Owner

so we can close this? :)

@rwstauner
Owner
@monken
Owner

we have to consider POD after DATA, there are a number of legit modules out there that put there pod in the DATA section (and sco supports this as well). Whoever browses the t/ directory probably knows that documentation in there is not indexed, i.e. not searchable. However, the Moose distribution has some good documentation in some tests where showing the NAME of the documentation actually makes sense.

@rwstauner
Owner
@monken monken closed this
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Something went wrong with that request. Please try again.