Skip to content

Commit 36ca717

Browse files
author
Josh Goldberg
authored
Added obsolete rule converters for tslint-microsoft-contrib rules (#1198)
1 parent 5c38d42 commit 36ca717

7 files changed

+54
-0
lines changed

src/converters/lintConfigs/rules/ruleConverters.ts

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -184,6 +184,7 @@ import { convertNoExplicitAny } from "./ruleConverters/no-explicit-any";
184184
import { convertNoFloatingPromises } from "./ruleConverters/no-floating-promises";
185185
import { convertNoForIn } from "./ruleConverters/no-for-in";
186186
import { convertNoForInArray } from "./ruleConverters/no-for-in-array";
187+
import { convertNoFunctionExpression } from "./ruleConverters/no-function-expression";
187188
import { convertNoImplicitDependencies } from "./ruleConverters/no-implicit-dependencies";
188189
import { convertNoImportSideEffect } from "./ruleConverters/no-import-side-effect";
189190
import { convertNoInferrableTypes } from "./ruleConverters/no-inferrable-types";
@@ -219,6 +220,7 @@ import { convertNoThisAssignment } from "./ruleConverters/no-this-assignment";
219220
import { convertNoTrailingWhitespace } from "./ruleConverters/no-trailing-whitespace";
220221
import { convertNoUnboundMethod } from "./ruleConverters/no-unbound-method";
221222
import { convertNoUnnecessaryClass } from "./ruleConverters/no-unnecessary-class";
223+
import { convertNoUnnecessaryFieldInitialization } from "./ruleConverters/no-unnecessary-field-initialization";
222224
import { convertNoUnnecessaryInitializer } from "./ruleConverters/no-unnecessary-initializer";
223225
import { convertNoUnnecessaryQualifier } from "./ruleConverters/no-unnecessary-qualifier";
224226
import { convertNoUnnecessarySemicolons } from "./ruleConverters/no-unnecessary-semicolons";
@@ -268,6 +270,7 @@ import { convertReactA11yRoleSupportsAriaProps } from "./ruleConverters/react-a1
268270
import { convertReactA11yTabIndexNoPositive } from "./ruleConverters/react-a11y-tabindex-no-positive";
269271
import { convertReactNoDangerousHtml } from "./ruleConverters/react-no-dangerous-html";
270272
import { convertReactTsxCurlySpacing } from "./ruleConverters/react-tsx-curly-spacing";
273+
import { convertReactUnusedPropsAndState } from "./ruleConverters/react-unused-props-and-state";
271274
import { convertRestrictPlusOperands } from "./ruleConverters/restrict-plus-operands";
272275
import { convertSemicolon } from "./ruleConverters/semicolon";
273276
import { convertSpaceBeforeFunctionParen } from "./ruleConverters/space-before-function-paren";
@@ -397,6 +400,7 @@ export const ruleConverters = new Map([
397400
["no-for-in-array", convertNoForInArray],
398401
["no-for-in", convertNoForIn],
399402
["no-forward-ref", convertNoForwardRef],
403+
["no-function-expression", convertNoFunctionExpression],
400404
["no-host-metadata-property", convertNoHostMetadataProperty],
401405
["no-identical-conditions", convertNoIdenticalConditions],
402406
["no-identical-expressions", convertNoIdenticalExpressions],
@@ -421,6 +425,7 @@ export const ruleConverters = new Map([
421425
["no-multiline-string-literals", convertNoMultilineStringLiterals],
422426
["no-multiline-string", convertNoMultilineString],
423427
["no-namespace", convertNoNamespace],
428+
["no-unnecessary-field-initialization", convertNoUnnecessaryFieldInitialization],
424429
["non-literal-fs-path", convertNonLiteralFsPath],
425430
["no-non-null-assertion", convertNoNonNullAssertion],
426431
["no-null-keyword", convertNoNullKeyword],
@@ -516,6 +521,7 @@ export const ruleConverters = new Map([
516521
["react-a11y-tabindex-no-positive", convertReactA11yTabIndexNoPositive],
517522
["react-no-dangerous-html", convertReactNoDangerousHtml],
518523
["react-tsx-curly-spacing", convertReactTsxCurlySpacing],
524+
["react-unused-props-and-state", convertReactUnusedPropsAndState],
519525
["relative-url-prefix", convertRelativeUrlPrefix],
520526
["restrict-plus-operands", convertRestrictPlusOperands],
521527
["rxjs-no-async-subscribe", convertNoAsyncSubscribe],
Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,5 @@
1+
import { RuleConverter } from "../ruleConverter";
2+
3+
export const convertNoFunctionExpression: RuleConverter = () => {
4+
return {};
5+
};
Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,5 @@
1+
import { RuleConverter } from "../ruleConverter";
2+
3+
export const convertNoUnnecessaryFieldInitialization: RuleConverter = () => {
4+
return {};
5+
};
Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,5 @@
1+
import { RuleConverter } from "../ruleConverter";
2+
3+
export const convertReactUnusedPropsAndState: RuleConverter = () => {
4+
return {};
5+
};
Lines changed: 11 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,11 @@
1+
import { convertNoFunctionExpression } from "../no-function-expression";
2+
3+
describe(convertNoFunctionExpression, () => {
4+
test("conversion without arguments", () => {
5+
const result = convertNoFunctionExpression({
6+
ruleArguments: [],
7+
});
8+
9+
expect(result).toEqual({});
10+
});
11+
});
Lines changed: 11 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,11 @@
1+
import { convertNoUnnecessaryFieldInitialization } from "../no-unnecessary-field-initialization";
2+
3+
describe(convertNoUnnecessaryFieldInitialization, () => {
4+
test("conversion without arguments", () => {
5+
const result = convertNoUnnecessaryFieldInitialization({
6+
ruleArguments: [],
7+
});
8+
9+
expect(result).toEqual({});
10+
});
11+
});
Lines changed: 11 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,11 @@
1+
import { convertReactUnusedPropsAndState } from "../react-unused-props-and-state";
2+
3+
describe(convertReactUnusedPropsAndState, () => {
4+
test("conversion without arguments", () => {
5+
const result = convertReactUnusedPropsAndState({
6+
ruleArguments: [],
7+
});
8+
9+
expect(result).toEqual({});
10+
});
11+
});

0 commit comments

Comments
 (0)