turn on rules for grouping exports at the end of modules in plain JS

Author: stekycz

src/flow.js

@@ -4,6 +4,8 @@ module.exports = {
 	],
 	rules: {
 		'no-duplicate-imports': 'off', // to support separate imports of types
+		'import/exports-last': 'off', // to support exporting types anywhere
+		'import/group-exports': 'off', // to support exporting types anywhere
 		'import/no-cycle': 'off', // to support importing types
 		'flowtype/array-style-complex-type': 'off',
 		'flowtype/array-style-simple-type': 'off',

src/import.js

@@ -16,10 +16,10 @@ module.exports = {
 		'import/default': 'error',
 		'import/dynamic-import-chunkname': 'off',
 		'import/export': 'error',
-		'import/exports-last': 'off',
+		'import/exports-last': 'error',
 		'import/extensions': ['error', 'never', {json: 'ignorePackages'}],
 		'import/first': 'error',
-		'import/group-exports': 'off',
+		'import/group-exports': 'error',
 		'import/imports-first': 'off', // deprecated, alias for "first"
 		'import/max-dependencies': 'off',
 		'import/named': 'error',

src/typescript.js

@@ -18,6 +18,8 @@ module.exports = {
 		},
 	],
 	rules: {
+		'import/exports-last': 'off', // to support exporting types/interfaces anywhere
+		'import/group-exports': 'off', // to support exporting types/interfaces anywhere
 		'import/no-cycle': 'off', // to support importing types/interfaces
 		'@typescript-eslint/adjacent-overload-signatures': 'error',
 		'@typescript-eslint/array-type': ['error', {

tests/flow.test.js

@@ -13,6 +13,8 @@ describe('Rules Configurations', () => {
 		const pluginRules = Object.keys(plugin.rules).map((ruleName) => `flowtype/${ruleName}`);
 		const overridenRuleNames = [
 			'no-duplicate-imports',
+			'import/exports-last',
+			'import/group-exports',
 			'import/no-cycle',
 		];
 

tests/typescript.test.js

@@ -19,6 +19,8 @@ describe('Rules Configurations', () => {
 			'quotes',
 			'require-await',
 			'semi',
+			'import/exports-last',
+			'import/group-exports',
 			'import/no-cycle',
 		];