refactor: eslint now orders imports

.eslintignore

@@ -1 +1,2 @@
 dist
+node_modules

.eslintrc.json

@@ -4,20 +4,24 @@
     "node": true
   },
   "parser": "@typescript-eslint/parser",
-  "plugins": ["@typescript-eslint", "prettier"],
+  "plugins": ["@typescript-eslint", "prettier", "import"],
   "extends": [
     "eslint:recommended",
     "plugin:@typescript-eslint/eslint-recommended",
     "plugin:@typescript-eslint/recommended",
-    "prettier/@typescript-eslint"
+    "prettier/@typescript-eslint",
+    "plugin:import/errors",
+    "plugin:import/warnings",
+    "plugin:import/typescript"
   ],
   "rules": {
     "prettier/prettier": "error",
     "semi": "error",
     "no-prototype-builtins": "off",
     "@typescript-eslint/no-var-requires": "off",
     "@typescript-eslint/explicit-function-return-type": "off",
-    "no-sparse-arrays": "off"
+    "no-sparse-arrays": "off",
+    "import/order": "error"
   },
   "overrides": [
     {
@@ -26,5 +30,8 @@
         "@typescript-eslint/no-non-null-assertion": "off"
       }
     }
-  ]
+  ],
+  "settings": {
+    "import/extensions": [".ts", ".js"]
+  }
 }

.prettierrc

@@ -1,6 +1,6 @@
 {
-    "singleQuote": true,
-    "tabWidth": 2,
-    "useTabs": false,
-    "printWidth": 120
+  "singleQuote": true,
+  "tabWidth": 2,
+  "useTabs": false,
+  "printWidth": 120
 }

package.json

@@ -6,7 +6,7 @@
   "scripts": {
     "test": "jest",
     "build": "tsc -p tsconfig.build.json",
-    "lint": "eslint ."
+    "lint": "eslint . --ext=\".js,.ts\""
   },
   "keywords": [
     "commitizen",
@@ -25,6 +25,7 @@
     "cz-conventional-changelog": "^3.0.2",
     "eslint": "^6.8.0",
     "eslint-config-prettier": "^6.9.0",
+    "eslint-plugin-import": "^2.20.1",
     "eslint-plugin-prettier": "^3.1.2",
     "husky": "^4.2.0",
     "jest": "^25.1.0",

src/build-scope.test.ts

@@ -1,5 +1,5 @@
-import { buildScope } from './build-scope';
 import { ListQuestion } from 'inquirer';
+import { buildScope } from './build-scope';
 
 describe('buildScope', () => {
   describe('when', () => {

src/build-scope.ts

@@ -1,7 +1,7 @@
 import { Rules } from '@commitlint/load';
 import { DistinctQuestion } from 'inquirer';
 import { whenFactory } from './when';
-import { validate, emptyValidator, maxLengthValidator, minLengthValidator, caseValidator } from './validators';
+import { caseValidator, emptyValidator, maxLengthValidator, minLengthValidator, validate } from './validators';
 import { wordCaseFilter } from './filters';
 
 export function buildScope(rules: Rules, questions: DistinctQuestion[]): DistinctQuestion[] {

src/engine.ts

@@ -1,19 +1,19 @@
 import { CommitlintConfig, Rules } from '@commitlint/load';
+import { DistinctQuestion, PromptModule } from 'inquirer';
+import { Commit } from 'commitizen';
 import { pipeWith } from './utils';
-import { PromptModule, DistinctQuestion } from 'inquirer';
-import { buildType } from './build-type';
+import { typeMaker } from './type-maker';
 import { buildBreakingChange } from './build-breaking-change';
 import { buildBody } from './build-body';
-import { subjectMaker } from './subect-maker';
 import { buildScope } from './build-scope';
-import { Commit } from 'commitizen';
+import { subjectMaker } from './subject-maker';
 
 function buildQuestions(rules: Rules) {
   const combinedQuestions = pipeWith<DistinctQuestion[]>(
     [],
-    x => buildType(rules, x),
+    x => typeMaker(x, rules),
     x => buildScope(rules, x),
-    x => subjectMaker(rules, x),
+    x => subjectMaker(x, rules),
     x => buildBody(rules, x),
     x => buildBreakingChange(rules, x)
   );

src/filters.test.ts

@@ -1,5 +1,5 @@
-import { leadingBlankFilter, fullStopFilter, wordCaseFilter, wordWrapFilter } from './filters';
 import { Level } from '@commitlint/load';
+import { fullStopFilter, leadingBlankFilter, wordCaseFilter, wordWrapFilter } from './filters';
 
 describe('filters', () => {
   describe('leadingBlankFilter', () => {

src/filters.ts

@@ -1,6 +1,6 @@
-import { Rule, Level, Case } from '@commitlint/load';
-import { wordCase } from './utils';
+import { Case, Level, Rule } from '@commitlint/load';
 import wordWrap from 'word-wrap';
+import { wordCase } from './utils';
 
 export function leadingBlankFilter(value: string, rule: Rule<void>): string {
   const [level, applicable] = rule;

src/subect-maker.ts → src/subject-maker.ts

@@ -1,8 +1,8 @@
 import { Rules } from '@commitlint/load';
-import { DistinctQuestion, Answers } from 'inquirer';
-import { validate, maxLengthValidator, emptyValidator, minLengthValidator, caseValidator } from './validators';
-import { wordCaseFilter, fullStopFilter } from './filters';
+import { Answers, DistinctQuestion } from 'inquirer';
 import { pipeWith, valueFromRule } from './utils';
+import { caseValidator, emptyValidator, maxLengthValidator, minLengthValidator, validate } from './validators';
+import { fullStopFilter, wordCaseFilter } from './filters';
 
 export function header(type: string, scope?: string, subject?: string): string {
   let header = `${type}`;
@@ -91,7 +91,7 @@ export function messageFactory(rules: Rules) {
   };
 }
 
-export function subjectMaker(rules: Rules, questions: DistinctQuestion[]): DistinctQuestion[] {
+export function subjectMaker(questions: DistinctQuestion[], rules: Rules): DistinctQuestion[] {
   const question: DistinctQuestion = {
     message: messageFactory(rules),
     name: 'subject',