Fixed issue with eslint-standalone rollup not removing all process.env checks.

solidjs-community / eslint-plugin-solid

Solid-specific linting rules for ESLint.

MIT License

209 stars 24 forks source link

Fixed issue with eslint-standalone rollup not removing all process.env checks. #65

Closed joshwilsonvu closed 1 year ago

joshwilsonvu commented 1 year ago

Having copy-pasted rollup-plugin-replace.js in for the standalone build, I didn't realize that each regex to test for replacement patterns like /process\.env\.DEBUG/ needed to have the g modifier on it to replace more than one occurrence per file. This led to a few process access slipping into the browser build and crashing the linter worker in playground.solidjs.com.