-
Notifications
You must be signed in to change notification settings - Fork 2k
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
eslint: fix no-undef for static properties #23382
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
Desktop breakage looks a little odd. Maybe we need to clear a cache?
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Works fine 🐑
Tested:
node_modules/.bin/eslint client/signup/step-wrapper/index.jsx
and got
The react/jsx-space-before-closing rule is deprecated. Please use the react/jsx-tag-spacing rule with the "beforeSelfClosing" option instead.
And also added a change like
static defaultProps = {
foo: 'bar',
};
and committed, which is similar to the original case where I found this, and got no issues.
@nosolosw tried to upgrade our eslint to 4.0 few months ago (around the GM) and had to revert it because of errors. And I'm pretty sure that it was something with class properties, too. Maybe exactly the same error. If we resolve this, could we try the eslint 4.0 upgrade again soon? |
* downgrade babel-eslint * shrinkwrap
I'm at my support rotation this week, so I've got limited bandwith. FWIW, #18106 is the issue I ran into. |
Note that |
@eliorivero today reported getting (false)
no-undef
warnings in eslint. It was caused by upgrading babel-eslint which has this bug in it. downgrading seems to fix it.To Test
npm install
node_modules/.bin/eslint client/signup/step-wrapper/index.jsx
. You shouldn't get any errors