-
-
Notifications
You must be signed in to change notification settings - Fork 78.8k
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
Add eslint-plugin-compat #23972
Add eslint-plugin-compat #23972
Conversation
'Opera >= 30' | ||
] | ||
} | ||
autoprefixer: {} |
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.
why did you let that line ? their are nothing in autoprefixer
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.
How will autoprefixer be included otherwise?
f24c4a2
to
912af1d
Compare
I ignored the |
912af1d
to
0213511
Compare
@Johann-S: what do we do with this? |
0213511
to
dbf9585
Compare
IMO we should merge this PR because currently folks PR aren't tested on Saucelabs so it will help folks to know if their code is compatible with our supported browsers |
This is so other plugins can use it too.
We can check for browser compatibility on lint time now.
dbf9585
to
57655e4
Compare
Note that we can move the browserlists config to a separate file; I went with package.json so that we don't have one more file.