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
Tune eslint packages test configuration #10848
Conversation
c401743
to
e9f6098
Compare
The failing test is related to this PR
@nicolo-ribaudo The test error is expected, because now I have added another test case when |
@@ -18,26 +18,23 @@ function verifyAndAssertMessagesWithSpecificESLint( | |||
node: true, | |||
es6: true, | |||
}, | |||
...overrideConfig, | |||
parserOptions: { | |||
sourceType, | |||
ecmaFeatures: { | |||
globalReturn: true, |
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.
Can we remove this? It's better to set it explicitly in the only test where we need it rather than having a default value.
The Travis failures are caused by network errors |
This PR is extracted from a messy working branch on the upcoming Yarn 2 support: https://github.com/JLHwung/babel/commits/standalone-rollup-bundle
As a follow up to #10705 , the integration test should not read root babel configuration.