Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Commit 915c87a

Browse files
jzolnowskiSplaktar
authored andcommitted
fix: configure import-order rule to sort imports based on package name
1 parent a9339a6 commit 915c87a

File tree

4 files changed

+2284
-1309
lines changed

4 files changed

+2284
-1309
lines changed

‎.eslintrc.json‎

Lines changed: 14 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -5,11 +5,10 @@
55
"commonjs": true
66
},
77
"globals": {
8-
"angular": true,
9-
"go": true
8+
"angular": true
109
},
11-
"plugins": ["prettier", "angular"], // https://github.com/Gillespie59/eslint-plugin-angular
12-
"extends": ["eslint:recommended", "plugin:prettier/recommended"],
10+
"plugins": ["prettier", "angular", "import-order"], // https://github.com/Gillespie59/eslint-plugin-angular
11+
"extends": ["eslint:recommended", "plugin:prettier/recommended", "plugin:import-order/recommended"],
1312
"rules": {
1413
"prettier/prettier": ["error"],
1514

@@ -208,7 +207,17 @@
208207
"yoda": "error",
209208

210209
// 24.15 https://eslint.org/docs/rules/max-lines.html
211-
"max-lines": ["error", 500]
210+
"max-lines": ["error", 500],
211+
"import-order/import-order": [
212+
"error",
213+
{
214+
"alphabetize": {
215+
"order": "desc",
216+
"caseInsensitive": false
217+
},
218+
"order": ["builtin", "external", "parent", "index", "sibling"]
219+
}
220+
]
212221

213222
// no rules for
214223

‎app/app.js‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,10 +1,10 @@
11
import 'angular-animate';
22
import 'angular-aria';
3-
import 'angular-messages';
43
import 'angular-material';
4+
import 'angular-messages';
5+
import { AppComponent } from './app.component';
56
import ComponentsModule from './components/components';
67
import ServicesModule from './services/services';
7-
import { AppComponent } from './app.component';
88

99
const appModule = angular
1010
.module('appModule', [

0 commit comments

Comments
(0)

AltStyle によって変換されたページ (->オリジナル) /