figure out rules with 'declare global' and non-shadowable globals

quick-lint / quick-lint-js

quick-lint-js finds bugs in JavaScript programs

https://quick-lint-js.com

GNU General Public License v3.0

1.52k stars 192 forks source link

figure out rules with 'declare global' and non-shadowable globals #1129

Open strager opened 8 months ago

strager commented 8 months ago

Imagine a global variable is declared as non-shadowable in quick-lint-js.config, and is also declared with 'declare global' in the source file. Should we treat that variable as shadowable? Or should we error on the 'declare global'?