Path ** does not recurse directories

[ghost]

I would expect leasot src/**/*.js to recurse directories within src and parse all js files

Example directory structure:

src
+-- todo-root.js
+-- folderA
       +-- todo-A-1.js
       +-- todo-A-2.js
+-- folderB
       +-- todo-B-1.js
       +-- todo-B-2.js
       +-- folderC
              +-- todo-C-1.js
              +-- todo-C-2.js

Correct Cases
Running leasot src/*.js parses files in src only, i.e. todo-root.js
Running leasot src/*/*/*.js parses files in folderC only

Incorrect Cases
Running leasot src/**/*.js parses files in folderA and the root of folderB only (It is missing todo-root.js as well as the files in folderC

[pgilad]

I agree with that what you suggested is the correct behavior for **/*.js parsing.

It also might be a problem with your shell, since leasot uses glob (https://github.com/pgilad/leasot/blob/master/lib/cli.js#L55) which should support this use case.

Can you try it with the following syntax:
leasot 'src/**/*.js' ? This will prevent your shell from expanding that glob.

[pgilad]

Let me know if I can help further

[ghost]

The issue still remains:

leasot src/**/*.js
leasot "src/**/*.js"
leasot 'src/**/*.js'

All of the above still do not recurse through sub-directories

[pgilad]

I've just tried to reproduce your error using the exact same file structure you mentioned (just called the root folder temp).
After running leasot this is what I get:

Also, if I run it with quotes around the glob pattern:

So I pretty much think this is an error with your OS or shell. Which do you use?

[ghost]

I can confirm it does infact work properly but quotes " ' are needed. Sorry for the false alarm!

[dkebler]

yup I missed this too but this gets me back to my recent issue #50 about excluding directories because if I try to do this from the root of my project it also searches node_modules until it crashes.
so yes on exclude directories, and maybe have a switch to exclude a .gitignore file would be cool