TrySound / rollup-plugin-eslint

This plugin in migrated https://github.com/rollup/plugins/tree/master/packages/eslint
MIT License
61 stars 17 forks source link

Issue with globals in eslintrc.* #31

Open KaboomFox opened 5 years ago

KaboomFox commented 5 years ago

The eslint CLI expects an array for globals while the eslintrc file expects a dictionary. This will case an error if you eslintrc file has Globals. https://github.com/TrySound/rollup-plugin-eslint/blob/master/index.js#L19

Reference to docs: https://github.com/eslint/eslint/blob/master/docs/developer-guide/nodejs-api.md

globals - An array of global variables to declare (default: empty array). Corresponds to --global, and similarly supports passing 'name:true' to denote a writeable global. Note: This differs from .eslintrc.* / baseConfig, where globals is an object.