-
Notifications
You must be signed in to change notification settings - Fork 26.3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Documentation for 'no-use-before-define' is missing #2708
Comments
I’d probably expect it in https://github.com/airbnb/javascript#hoisting. |
@ljharb let me know where you should want to put it in |
@MohitBansal321 I think we actually need a new 14.5 talking about why relying on hoisting should be avoided. |
so this rule is only applicable to the new 14.5? @ljharb |
Yes, because the guide doesn't actually discuss not relying on hoisting, but the config has been enforcing that for a very long time. |
Could you please let me know if this issue is still open for contributions? If so, I would be very interested in working on it. Thank you! |
@Vivek-04022001 please stop commenting that; if it's open, has the PR wanted label, and has no linked PR, then it's always open for contributions, and the way to "claim" it is to put up a PR that links to it. |
The rule is set to error in the config:
javascript/packages/eslint-config-airbnb-base/rules/variables.js
Line 54 in 1eadb93
But there's no mention of it in the guide:
https://github.com/airbnb/javascript/blob/master/README.md
This rule should be mentioned under the Variables section.
The text was updated successfully, but these errors were encountered: