fix configuration for disabling duplicate imports

Author: stekycz

src/flow.js

@@ -3,6 +3,7 @@ module.exports = {
 		'flowtype',
 	],
 	rules: {
+		'no-duplicate-imports': 'off', // to support separate imports of types
 		'flowtype/array-style-complex-type': 'off',
 		'flowtype/array-style-simple-type': 'off',
 		'flowtype/arrow-parens': ['error', 'always'],

src/native.js

@@ -125,7 +125,7 @@ module.exports = {
 		'no-dupe-class-members': 'error',
 		'no-dupe-keys': 'error',
 		'no-duplicate-case': 'error',
-		'no-duplicate-imports': 'off',
+		'no-duplicate-imports': 'error',
 		'no-else-return': ['error', {
 			allowElseIf: false,
 		}],

src/typescript.js

@@ -18,7 +18,6 @@ module.exports = {
 		},
 	],
 	rules: {
-		'no-duplicate-imports': 'error',
 		'@typescript-eslint/adjacent-overload-signatures': 'error',
 		'@typescript-eslint/array-type': ['error', {
 			default: 'array-simple',

tests/flow.test.js

@@ -11,6 +11,9 @@ describe('Rules Configurations', () => {
 	describe('flow', () => {
 		const localRules = Object.keys(localConfig.rules);
 		const pluginRules = Object.keys(plugin.rules).map((ruleName) => `flowtype/${ruleName}`);
+		const overridenRuleNames = [
+			'no-duplicate-imports',
+		];
 
 		it('has configuration for all supported rules', () => {
 			const notConfiguredPluginRules = pluginRules.filter((ruleName) => !localRules.includes(ruleName));
@@ -19,7 +22,9 @@ describe('Rules Configurations', () => {
 		});
 
 		it('does not have configuration for not existing rules', () => {
-			const extraLocalRules = localRules.filter((ruleName) => !pluginRules.includes(ruleName));
+			const extraLocalRules = localRules
+				.filter((ruleName) => !pluginRules.includes(ruleName))
+				.filter((ruleName) => !overridenRuleNames.includes(ruleName));
 
 			expect(extraLocalRules).toEqual([]);
 		});

tests/typescript.test.js

@@ -15,7 +15,6 @@ describe('Rules Configurations', () => {
 			'brace-style',
 			'func-call-spacing',
 			'indent',
-			'no-duplicate-imports',
 			'no-empty-function',
 			'quotes',
 			'require-await',