-
Notifications
You must be signed in to change notification settings - Fork 24.3k
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
switch from babel-eslint to hermes-eslint for flow code
Summary: `hermes-eslint` is built by Meta to work with the latest Flow code. It follows the latest ESLint standards and AST, and has a true scope analyser to ensure best compatibility with core ESLint rules. Reviewed By: motiz88 Differential Revision: D37181192 fbshipit-source-id: 1f59e01f306792e67a4977435c5c77e0000d960a
- Loading branch information
1 parent
577582e
commit 7b0ba6d
Showing
4 changed files
with
99 additions
and
60 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,77 @@ | ||
| /** | ||
| * Copyright (c) Meta Platforms, Inc. and affiliates. | ||
| * | ||
| * This source code is licensed under the MIT license found in the | ||
| * LICENSE file in the root directory of this source tree. | ||
| * | ||
| * @format | ||
| */ | ||
|
|
||
| 'use strict'; | ||
|
|
||
| module.exports = { | ||
| root: true, | ||
|
|
||
| extends: ['./packages/eslint-config-react-native-community/index.js'], | ||
|
|
||
| plugins: ['@react-native/eslint-plugin-specs'], | ||
|
|
||
| overrides: [ | ||
| // overriding the JS config from eslint-config-react-native-community config to ensure | ||
| // that we use hermes-eslint for all js files | ||
| { | ||
| files: ['*.js'], | ||
| parser: 'hermes-eslint', | ||
| rules: { | ||
| // These rules are not required with hermes-eslint | ||
| 'ft-flow/define-flow-type': 0, | ||
| 'ft-flow/use-flow-type': 0, | ||
| 'flowtype/define-flow-type': 0, | ||
| 'flowtype/use-flow-type': 0, | ||
| // flow handles this check for us, so it's not required | ||
| 'no-undef': 0, | ||
| }, | ||
| }, | ||
|
|
||
| { | ||
| files: ['Libraries/**/*.js'], | ||
| rules: { | ||
| '@react-native-community/no-haste-imports': 2, | ||
| '@react-native-community/error-subclass-name': 2, | ||
| '@react-native-community/platform-colors': 2, | ||
| '@react-native/specs/react-native-modules': 2, | ||
| }, | ||
| }, | ||
| { | ||
| files: ['flow-typed/**/*.js'], | ||
| rules: { | ||
| quotes: 0, | ||
| }, | ||
| }, | ||
| { | ||
| files: [ | ||
| '**/__fixtures__/**/*.js', | ||
| '**/__mocks__/**/*.js', | ||
| '**/__tests__/**/*.js', | ||
| 'jest/**/*.js', | ||
| 'packages/rn-tester/**/*.js', | ||
| ], | ||
| globals: { | ||
| // Expose some Jest globals for test helpers | ||
| afterAll: true, | ||
| afterEach: true, | ||
| beforeAll: true, | ||
| beforeEach: true, | ||
| expect: true, | ||
| jest: true, | ||
| }, | ||
| }, | ||
| { | ||
| files: ['**/__tests__/**/*-test.js'], | ||
| env: { | ||
| jasmine: true, | ||
| jest: true, | ||
| }, | ||
| }, | ||
| ], | ||
| }; |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters