Enable grouped-imports for ordered-imports rule in tslint:all config (#…

…4420)

src/configs/all.ts

@@ -227,6 +227,7 @@ export const rules = {
     "ordered-imports": [
         true,
         {
+            "grouped-imports": true,
             "import-sources-order": "case-insensitive",
             "named-imports-order": "case-insensitive",
             "module-source-path": "full",

src/configuration.ts

@@ -20,8 +20,8 @@ import * as yaml from "js-yaml";
 import { Minimatch } from "minimatch";
 import * as os from "os";
 import * as path from "path";
-import { FatalError, showWarningOnce } from "./error";
 
+import { FatalError, showWarningOnce } from "./error";
 import { IOptions, RuleSeverity } from "./language/rule/rule";
 import { findRule } from "./ruleLoader";
 import { arrayify, hasOwnProperty, stripComments, tryResolvePackage } from "./utils";

src/enableDisableRules.ts

@@ -19,6 +19,7 @@
 
 import * as utils from "tsutils";
 import * as ts from "typescript";
+
 import { RuleFailure } from "./language/rule/rule";
 
 /**

src/formatterLoader.ts

@@ -17,6 +17,7 @@
 
 import * as fs from "fs";
 import * as path from "path";
+
 import { FormatterConstructor } from "./index";
 import { camelize, tryResolvePackage } from "./utils";
 

src/formatters/checkstyleFormatter.ts

@@ -18,7 +18,6 @@
 import { AbstractFormatter } from "../language/formatter/abstractFormatter";
 import { IFormatterMetadata } from "../language/formatter/formatter";
 import { RuleFailure } from "../language/rule/rule";
-
 import * as Utils from "../utils";
 
 export class Formatter extends AbstractFormatter {

src/formatters/codeFrameFormatter.ts

@@ -15,13 +15,12 @@
  * limitations under the License.
  */
 
-import { AbstractFormatter } from "../language/formatter/abstractFormatter";
-import { IFormatterMetadata } from "../language/formatter/formatter";
-import { RuleFailure } from "../language/rule/rule";
-
 import codeFrame = require("babel-code-frame");
 import chalk from "chalk";
 
+import { AbstractFormatter } from "../language/formatter/abstractFormatter";
+import { IFormatterMetadata } from "../language/formatter/formatter";
+import { RuleFailure } from "../language/rule/rule";
 import * as Utils from "../utils";
 
 export class Formatter extends AbstractFormatter {

src/formatters/jsonFormatter.ts

@@ -18,7 +18,6 @@
 import { AbstractFormatter } from "../language/formatter/abstractFormatter";
 import { IFormatterMetadata } from "../language/formatter/formatter";
 import { RuleFailure } from "../language/rule/rule";
-
 import * as Utils from "../utils";
 
 export class Formatter extends AbstractFormatter {

src/formatters/junitFormatter.ts

@@ -18,7 +18,6 @@
 import { AbstractFormatter } from "../language/formatter/abstractFormatter";
 import { IFormatterMetadata } from "../language/formatter/formatter";
 import { RuleFailure } from "../language/rule/rule";
-
 import * as Utils from "../utils";
 
 export class Formatter extends AbstractFormatter {

src/formatters/pmdFormatter.ts

@@ -18,7 +18,6 @@
 import { AbstractFormatter } from "../language/formatter/abstractFormatter";
 import { IFormatterMetadata } from "../language/formatter/formatter";
 import { RuleFailure } from "../language/rule/rule";
-
 import * as Utils from "../utils";
 
 export class Formatter extends AbstractFormatter {

src/formatters/stylishFormatter.ts

@@ -15,12 +15,11 @@
  * limitations under the License.
  */
 
+import chalk from "chalk";
+
 import { AbstractFormatter } from "../language/formatter/abstractFormatter";
 import { IFormatterMetadata } from "../language/formatter/formatter";
 import { RuleFailure } from "../language/rule/rule";
-
-import chalk from "chalk";
-
 import * as Utils from "../utils";
 
 export class Formatter extends AbstractFormatter {

src/formatters/tapFormatter.ts

@@ -18,7 +18,6 @@
 import { AbstractFormatter } from "../language/formatter/abstractFormatter";
 import { IFormatterMetadata } from "../language/formatter/formatter";
 import { RuleFailure } from "../language/rule/rule";
-
 import * as Utils from "../utils";
 
 export class Formatter extends AbstractFormatter {

src/formatters/vsoFormatter.ts

@@ -18,7 +18,6 @@
 import { AbstractFormatter } from "../language/formatter/abstractFormatter";
 import { IFormatterMetadata } from "../language/formatter/formatter";
 import { RuleFailure } from "../language/rule/rule";
-
 import * as Utils from "../utils";
 
 export class Formatter extends AbstractFormatter {

src/language/formatter/abstractFormatter.ts

@@ -16,6 +16,7 @@
  */
 
 import { RuleFailure } from "../rule/rule";
+
 import { IFormatter, IFormatterMetadata } from "./formatter";
 
 export abstract class AbstractFormatter implements IFormatter {

src/language/rule/abstractRule.ts

@@ -18,6 +18,7 @@
 import * as ts from "typescript";
 
 import { IWalker, WalkContext } from "../walker";
+
 import { IOptions, IRule, IRuleMetadata, RuleFailure, RuleSeverity } from "./rule";
 
 export type NoInfer<T> = T & { [K in keyof T]: T[K] };

src/language/rule/typedRule.ts

@@ -18,6 +18,7 @@
 import * as ts from "typescript";
 
 import { showWarningOnce } from "../../error";
+
 import { AbstractRule } from "./abstractRule";
 import { ITypedRule, RuleFailure } from "./rule";
 

src/language/walker/blockScopeAwareRuleWalker.ts

@@ -19,6 +19,7 @@ import * as ts from "typescript";
 
 import { IOptions } from "../rule/rule";
 import { isBlockScopeBoundary } from "../utils";
+
 import { ScopeAwareRuleWalker } from "./scopeAwareRuleWalker";
 
 // tslint:disable:deprecation (extends deprecated class and uses deprecated utils - doesn't matter because it's deprecated, too)

src/language/walker/programAwareRuleWalker.ts

@@ -18,6 +18,7 @@
 import * as ts from "typescript";
 
 import { IOptions } from "../rule/rule";
+
 import { RuleWalker } from "./ruleWalker";
 
 export class ProgramAwareRuleWalker extends RuleWalker {

src/language/walker/ruleWalker.ts

@@ -18,6 +18,7 @@
 import * as ts from "typescript";
 
 import { Fix, IOptions, Replacement, RuleFailure } from "../rule/rule";
+
 import { SyntaxWalker } from "./syntaxWalker";
 import { IWalker } from "./walker";
 

src/language/walker/scopeAwareRuleWalker.ts

@@ -19,6 +19,7 @@ import * as ts from "typescript";
 
 import { IOptions } from "../rule/rule";
 import { isScopeBoundary } from "../utils";
+
 import { RuleWalker } from "./ruleWalker";
 
 /**

src/language/walker/walker.ts

@@ -18,6 +18,7 @@
 import * as ts from "typescript";
 
 import { RuleFailure } from "../rule/rule";
+
 import { WalkContext } from "./walkContext";
 
 export interface IWalker {

src/rules/arrowReturnShorthandRule.ts

@@ -20,6 +20,7 @@ import * as ts from "typescript";
 
 import * as Lint from "../index";
 import { hasCommentAfterPosition } from "../language/utils";
+
 import { codeExamples } from "./code-examples/arrowReturnShorthand.examples";
 
 const OPTION_MULTILINE = "multiline";

src/rules/awaitPromiseRule.ts

@@ -23,6 +23,7 @@ import {
     isUnionOrIntersectionType,
 } from "tsutils";
 import * as ts from "typescript";
+
 import * as Lint from "../index";
 
 export class Rule extends Lint.Rules.TypedRule {

src/rules/banTsIgnoreRule.ts

@@ -17,7 +17,9 @@
 
 import { forEachComment } from "tsutils";
 import * as ts from "typescript";
+
 import * as Lint from "../index";
+
 import { codeExamples } from "./code-examples/banTsIgnore.examples";
 
 export class Rule extends Lint.Rules.AbstractRule {

src/rules/classNameRule.ts

@@ -20,6 +20,7 @@ import * as ts from "typescript";
 
 import * as Lint from "../index";
 import { isPascalCased } from "../utils";
+
 import { codeExamples } from "./code-examples/className.examples";
 
 export class Rule extends Lint.Rules.AbstractRule {

src/rules/completed-docs/blockExclusion.ts

@@ -19,6 +19,7 @@ import { hasModifier } from "tsutils";
 import * as ts from "typescript";
 
 import { ALL, Visibility, VISIBILITY_EXPORTED, VISIBILITY_INTERNAL } from "../completedDocsRule";
+
 import { Exclusion } from "./exclusion";
 
 export interface IBlockExclusionDescriptor {

src/rules/completed-docs/classExclusion.ts

@@ -28,6 +28,7 @@ import {
     PRIVACY_PROTECTED,
     PRIVACY_PUBLIC,
 } from "../completedDocsRule";
+
 import { Exclusion } from "./exclusion";
 
 export interface IClassExclusionDescriptor {

src/rules/completed-docs/exclusion.ts

@@ -18,6 +18,7 @@
 import * as ts from "typescript";
 
 import { All, ALL } from "../completedDocsRule";
+
 import { ExclusionDescriptor } from "./exclusionDescriptors";
 
 export abstract class Exclusion<TDescriptor extends ExclusionDescriptor> {

src/rules/completed-docs/exclusionDescriptors.ts

@@ -16,6 +16,7 @@
  */
 
 import { DocType } from "../completedDocsRule";
+
 import { IBlockExclusionDescriptor } from "./blockExclusion";
 import { IClassExclusionDescriptor } from "./classExclusion";
 import { ITagExclusionDescriptor } from "./tagExclusion";

src/rules/completed-docs/exclusionFactory.ts

@@ -17,6 +17,7 @@
 
 import { hasOwnProperty } from "../../utils";
 import { DocType } from "../completedDocsRule";
+
 import { BlockExclusion, IBlockExclusionDescriptor } from "./blockExclusion";
 import { ClassExclusion, IClassExclusionDescriptor } from "./classExclusion";
 import { Exclusion } from "./exclusion";

src/rules/completedDocsRule.ts

@@ -18,6 +18,7 @@
 import * as ts from "typescript";
 
 import * as Lint from "../index";
+
 import { IInputExclusionDescriptors } from "./completed-docs/exclusionDescriptors";
 import { ExclusionFactory, ExclusionsMap } from "./completed-docs/exclusionFactory";
 

src/rules/curlyRule.ts

@@ -17,7 +17,9 @@
 
 import { isBlock, isIfStatement, isIterationStatement, isSameLine } from "tsutils";
 import * as ts from "typescript";
+
 import * as Lint from "../index";
+
 import { codeExamples } from "./code-examples/curly.examples";
 
 const OPTION_AS_NEEDED = "as-needed";

src/rules/cyclomaticComplexityRule.ts

@@ -17,6 +17,7 @@
 
 import { isIdentifier } from "tsutils";
 import * as ts from "typescript";
+
 import * as Lint from "../index";
 import { isFunctionScopeBoundary } from "../utils";
 

src/rules/deprecationRule.ts

@@ -32,6 +32,7 @@ import {
     isVariableDeclarationList,
 } from "tsutils";
 import * as ts from "typescript";
+
 import * as Lint from "../index";
 
 export class Rule extends Lint.Rules.TypedRule {

src/rules/fileHeaderRule.ts

@@ -16,6 +16,7 @@
  */
 
 import * as ts from "typescript";
+
 import * as Lint from "../index";
 
 const ENFORCE_TRAILING_NEWLINE = "enforce-trailing-newline";

src/rules/functionConstructorRule.ts

@@ -19,6 +19,7 @@ import { isCallExpression, isIdentifier, isNewExpression } from "tsutils";
 import * as ts from "typescript";
 
 import * as Lint from "..";
+
 import { codeExamples } from "./code-examples/functionConstructor.examples";
 
 export class Rule extends Lint.Rules.AbstractRule {

src/rules/importBlacklistRule.ts

@@ -23,6 +23,7 @@ import {
     isNamedImports,
 } from "tsutils";
 import * as ts from "typescript";
+
 import * as Lint from "../index";
 
 export class Rule extends Lint.Rules.AbstractRule {

src/rules/interfaceOverTypeLiteralRule.ts

@@ -17,6 +17,7 @@
 
 import { getChildOfKind, isTypeAliasDeclaration, isTypeLiteralNode } from "tsutils";
 import * as ts from "typescript";
+
 import * as Lint from "../index";
 
 export class Rule extends Lint.Rules.AbstractRule {

src/rules/maxClassesPerFileRule.ts

@@ -17,6 +17,7 @@
 
 import { isClassDeclaration, isClassExpression } from "tsutils";
 import * as ts from "typescript";
+
 import * as Lint from "../index";
 
 interface Options {

src/rules/maxFileLineCountRule.ts

@@ -16,6 +16,7 @@
  */
 
 import * as ts from "typescript";
+
 import * as Lint from "../index";
 
 export class Rule extends Lint.Rules.AbstractRule {

src/rules/maxLineLengthRule.ts

@@ -17,6 +17,7 @@
 
 import { getLineRanges } from "tsutils";
 import * as ts from "typescript";
+
 import * as Lint from "../index";
 
 interface MaxLineLengthRuleOptions {

src/rules/newlineBeforeReturnRule.ts

@@ -17,6 +17,7 @@
 
 import { getPreviousStatement } from "tsutils";
 import * as ts from "typescript";
+
 import * as Lint from "../index";
 
 export class Rule extends Lint.Rules.AbstractRule {

src/rules/newlinePerChainedCallRule.ts

@@ -22,6 +22,7 @@ import {
     isSameLine,
 } from "tsutils";
 import * as ts from "typescript";
+
 import * as Lint from "..";
 
 export class Rule extends Lint.Rules.AbstractRule {

src/rules/noAnyRule.ts

@@ -18,6 +18,7 @@
 import * as ts from "typescript";
 
 import * as Lint from "../index";
+
 import { codeExamples } from "./code-examples/noAny.examples";
 
 export class Rule extends Lint.Rules.AbstractRule {

src/rules/noDefaultImportRule.ts

@@ -17,6 +17,7 @@
 
 import { isImportDeclaration, isNamedImports, isStringLiteral } from "tsutils";
 import * as ts from "typescript";
+
 import * as Lint from "../index";
 
 const fromModulesConfigOptionName = "fromModules";

src/rules/noDuplicateImportsRule.ts

@@ -17,6 +17,7 @@
 
 import { isImportDeclaration, isLiteralExpression, isModuleDeclaration } from "tsutils";
 import * as ts from "typescript";
+
 import * as Lint from "../index";
 
 export class Rule extends Lint.Rules.AbstractRule {

src/rules/noEmptyInterfaceRule.ts

@@ -19,6 +19,7 @@ import { isInterfaceDeclaration } from "tsutils";
 import * as ts from "typescript";
 
 import * as Lint from "../index";
+
 import { codeExamples } from "./code-examples/noEmptyInterface.examples";
 
 export class Rule extends Lint.Rules.AbstractRule {

src/rules/noInferredEmptyObjectTypeRule.ts

@@ -17,6 +17,7 @@
 
 import { isObjectFlagSet, isObjectType, isTypeReference } from "tsutils";
 import * as ts from "typescript";
+
 import * as Lint from "../index";
 
 export class Rule extends Lint.Rules.TypedRule {