Solved issue of traversing into child directories

[blakeembrey]

Read through the node-glob code for a bit and found that the suffix of /** does what we need and excludes the whole directory properly from processing (also works for files!). Might be relevant to everyone implementing tsconfig.json implementations, and not using this module.

Tested by logging before https://github.com/isaacs/node-glob/blob/master/glob.js#L333-L334.

Before:

. false
. false
. false
cwd.ts false
cwd.ts false
excluded false
excluded false
included false
included false
tsconfig.json false
tsconfig.json false
. false
cwd.ts false
cwd.ts false
excluded false
excluded false
included false
included false
tsconfig.json false
tsconfig.json false
excluded false
excluded/foo.ts false
excluded/foo.ts false
excluded false
excluded/foo.ts false
excluded/foo.ts false
included false
included/foo.ts false
included/foo.ts false
included false
included/foo.ts false
included/foo.ts false

After:

. false
. false
. false
cwd.ts true
cwd.ts true
excluded true
excluded true
included false
included false
tsconfig.json false
tsconfig.json false
. false
cwd.ts true
cwd.ts true
excluded true
excluded true
included false
included false
tsconfig.json false
tsconfig.json false
included false
included/foo.ts false
included/foo.ts false
included false
included/foo.ts false
included/foo.ts false

/cc @basarat

[nycdotnet]

I haven't looked at this beyond your description, but I'm pleased you've found this result.

[blakeembrey]

@nycdotnet Haha, yeah - I was just about to comment on your other mention about exclude. This might be relevant to you if you're using node-glob.

Reference: TypeStrong/atom-typescript#634

[basarat]

that the suffix of /** does what we need and excludes the whole directory properly from processing (also works for files!)

Phew. Awesome thanks! I'll start sorting out the other places :) 🌹