Add enable option

[binarykitchen]

That's the code I have now, and don't like it:

      plugins: [
        pluginStylus(),
        ...
        isProd() ? pluginGoogleAnalytics({ id: getGoogleTagID() }) : undefined, // <------------- This yuck
      ],

Creates an empty element in the plugin array. Could you add a new option enabled, similar like your other eslint plugin?

pluginEslint({
  enable: process.env.NODE_ENV === "production",
});

That would solve it elegantly. Thanks.

[chenjiahan]

How about using &&?

      plugins: [
        isProd() && pluginGoogleAnalytics({ id: getGoogleTagID() }),
      ],

[binarykitchen]

Well, that's not better either, it produces [false] when not in production mode.

[chenjiahan]

The plugins array can accept false

[chenjiahan]

I'm open to adding an enable option and welcome PR

[binarykitchen]

I think it's important to keep all your plugins consistent. So that they all have the same pattern. Enable/Disable, depending on the environment.

(I'm a family father and for doing a PR, I'd have to examine code for which I do not have the time right now)

[binarykitchen]

Thanks!