Fixes #6726 ignore config option

[subtleGradient]

https://phabricator.babeljs.io/T6726

[codecov-io]

Current coverage is 88.13% (diff: 100%)

Merging #3644 into master will not change coverage

@@             master      #3644   diff @@
==========================================
  Files           188        188          
  Lines         10924      10924          
  Methods        1162       1162          
  Messages          0          0          
  Branches       2492       2492          
==========================================
  Hits           9628       9628          
  Misses         1296       1296          
  Partials          0          0          

Powered by Codecov. Last update a2d66c0...87f7d04

[STRML]

This is a really annoying bug and contradicts the docs. Could someone please review and merge it ASAP?

[quantizor]

@hzoo ?

[hzoo]

So sorry for the delay, I'm not particularly familiar with this of the codebase but @kittens says it was probably an artifact of some older code at the time. 67201e9

I think it's leftover from when babel.transform didn't actually return if a file was ignroed
so if we left it up to babel.transform then we'd just return the code verbatim and wouldn't do anything to it and we'd write it to the file system

[gcedo]

I am continuing to experience this bug, babel-cli@6.14.0. Now the ignore option is supported, but files are not ignored recursively.

E.g., declaring:

"ignore": [
        "node_modules"
]

Will cause the build to fail anyway because, as babel will try to transpile files deep down in the node_modules hierarchy:

69% 1206/1215 build modulesnode_modules/babel-plugin-react-transform/lib/index.js -> dist/node_modules/babel-plugin-react-transform/lib/index.js

[danez]

@gcedo Does it work if you specify the option on the command line?

babel --ignore 'node_modules' ... 

[gcedo]

Yes, it does. As I have seen the bug fix I have moved the options in .babelrc, but now it fails 😢

[danez]

Can you please open a new issue, so we do not forget about it. Thanks

[pdillon]

@danez Opened #4707 and corresponding PR #4708

[cula]

Happen again on babel 6.24.0

[thabti]

happens on 6.26.0