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

match 'other files' by path, not filename #425

Merged
merged 2 commits into from Oct 27, 2015

Conversation

haarg
Copy link
Member

@haarg haarg commented Oct 19, 2015

The list of files given as 'other' are meant to be excluded from
indexing in the root of a dist. Some may intentionally exist in
subdirectories (particularly in lib/) and meant to be indexed. Instead
of matching based on the filename, match based on the full path so it
only applies to files in the root directory (so far).

Fixes metacpan/metacpan-web#1576

The list of files given as 'other' are meant to be excluded from
indexing in the root of a dist.  Some may intentionally exist in
subdirectories (particularly in lib/) and meant to be indexed.  Instead
of matching based on the filename, match based on the full path so it
only applies to files in the root directory (so far).
@oalders
Copy link
Member

oalders commented Oct 27, 2015

👍

oalders added a commit that referenced this pull request Oct 27, 2015
match 'other files' by path, not filename
@oalders oalders merged commit 4ab3bc1 into metacpan:master Oct 27, 2015
@haarg haarg deleted the other-via-path branch July 12, 2016 18:12
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.

None yet

2 participants