Changed the order files get checked for ignoring

[N0NamedGuy]

This allows the plugin to have greater control on what ESLint gets to see and not.

For more information see #37

[RobbieTheWagner]

@TrySound can we please merge this, so this plugin is not broken?

[atti187]

Same thing goes for https://github.com/airbnb/react-dates which ships with an .eslintrc extending eslint-config-airbnb (which is a dev dependency of theirs). This results in node_modules\react-dates\initialize.js Error: Failed to load config "airbnb" to extend from..

My take is that if the exclude pattern includes node_modules/**, it shouldn't even bother to check for further config related to the file in question.

As a workaround, I monkey-patched the plugin;

const { createFilter } = require('rollup-pluginutils');
const eslintPlugin = (options = {}) => {
    const filter = createFilter(options.include, options.exclude || /node_modules/);

    const original = eslint(options);
    const originalTransform = original.transform;

    original.transform = (code, id) => {
        if (!filter(id)) {
            return null;
        } else {
            return originalTransform(code, id);
        }
    };

    return original;
};

[RobbieTheWagner]

@TrySound are you interested in maintaining this anymore? If not, could you transfer it or should we fork it? This has been open almost a year.

[TrySound]

Feel free to migrate it to https://github.com/rollup/plugins monorepo. I will add deprecation to this package then.

[RobbieTheWagner]

@TrySound I wouldn't know how to do that. I don't do monorepos. If someone can do that, please feel free, but otherwise all I know to do is fork and release a separate package.

[atti187]

@TrySound I've made a PR at rollup/plugins#575. Please have a look...