no-use-before-define with global variable reassign

Hi, no-use-before-define triggers an error for this kind of code:

var global = global || 'default';

The variable is defined in the globals section of the eslint file. So in my opinion, it shouldn’t trigger the error.

What do you think?

My environment

• ESLint Version: 4.2.0
• Node Version: 8.9.1
• npm Version: 5.5.1
• default parser

[kaicataldo]

I think this is actually the correct behavior. When it’s being redeclared, global is actually undefined because it is now referencing the variable that was just declared. When it’s reassigned, it references the previously declared variable in the outer/global scope. Check out this quick illustration I put together in JS Bin.

[eslint-deprecated[bot]]

Unfortunately, it looks like there wasn’t enough interest from the team or community to implement this change. While we wish we’d be able to accommodate everyone’s requests, we do need to prioritize. We’ve found that issues failing to reach accepted status after 21 days tend to never be accepted, and as such, we close those issues. This doesn’t mean the idea isn’t interesting or useful, just that it’s not something the team can commit to.

Thanks for contributing to ESLint and we appreciate your understanding.