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

Do not scan first-level dirs starting with . #1555

Merged

Conversation

@dondelelcaro
Copy link
Contributor

commented Aug 26, 2016

  • Currently the code skips directories with level > 1 starting with .,
    but does not skip first-level directories starting with .; fix this
    by matching for files/directories which start with . in addition to
    those that contain /.
Do not scan first-level dirs starting with .
 - Currently the code skips directories with level > 1 starting with .,
   but does not skip first-level directories starting with .; fix this
   by matching for files/directories which start with . in addition to
   those that contain /.

@adamcik adamcik merged commit bf24875 into mopidy:develop Oct 2, 2016

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
@adamcik

This comment has been minimized.

Copy link
Member

commented Oct 2, 2016

Thanks for the patch, I was about to ask you to add a test for this, but I see now that we don't actually have any tests for the command :( I've filed #1564 to address this.

adamcik added a commit that referenced this pull request Oct 2, 2016

docs: Add PR#1555 to changelog
Note that #1555 could also have been a candidate for a point release.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
2 participants
You can’t perform that action at this time.