Skip to content

Commit 42b1821

Browse files
dimaboryJosh Goldberg
authored and
Josh Goldberg
committed
fix(reporting): "do not have an ESLint equivalent" is misleading #299 (#308)
1 parent 43ae0fc commit 42b1821

File tree

4 files changed

+9
-8
lines changed

4 files changed

+9
-8
lines changed

src/reporting/reportConversionResults.test.ts

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -172,7 +172,7 @@ describe("reportConversionResults", () => {
172172
);
173173
expectEqualWrites(
174174
logger.info.write,
175-
"tslint-rule-one does not yet have an ESLint equivalent.",
175+
'tslint-to-eslint-config does not know the ESLint equivalent for TSLint\'s "tslint-rule-one"',
176176
);
177177
});
178178

@@ -206,8 +206,8 @@ describe("reportConversionResults", () => {
206206
);
207207
expectEqualWrites(
208208
logger.info.write,
209-
"tslint-rule-one does not yet have an ESLint equivalent.",
210-
"tslint-rule-two does not yet have an ESLint equivalent.",
209+
'tslint-to-eslint-config does not know the ESLint equivalent for TSLint\'s "tslint-rule-one"',
210+
'tslint-to-eslint-config does not know the ESLint equivalent for TSLint\'s "tslint-rule-two"',
211211
);
212212
});
213213

src/reporting/reportConversionResults.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -29,7 +29,7 @@ export const reportConversionResults = (
2929
logMissingConversionTarget(
3030
"rule",
3131
(setting: TSLintRuleOptions) =>
32-
`${setting.ruleName} does not yet have an ESLint equivalent.${EOL}`,
32+
`tslint-to-eslint-config does not know the ESLint equivalent for TSLint's "${setting.ruleName}"${EOL}`,
3333
ruleConversionResults.missing,
3434
dependencies.logger,
3535
[

src/reporting/reportEditorSettingConversionResults.test.ts

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -126,7 +126,7 @@ describe("reportEditorSettingConversionResults", () => {
126126
);
127127
expectEqualWrites(
128128
logger.info.write,
129-
"tslint-editor-setting-one does not yet have an ESLint equivalent.",
129+
'tslint-to-eslint-config does not know the ESLint equivalent for TSLint\'s "tslint-editor-setting-one"',
130130
);
131131
});
132132

@@ -156,8 +156,8 @@ describe("reportEditorSettingConversionResults", () => {
156156
);
157157
expectEqualWrites(
158158
logger.info.write,
159-
"tslint-editor-setting-one does not yet have an ESLint equivalent.",
160-
"tslint-editor-setting-two does not yet have an ESLint equivalent.",
159+
'tslint-to-eslint-config does not know the ESLint equivalent for TSLint\'s "tslint-editor-setting-one"',
160+
'tslint-to-eslint-config does not know the ESLint equivalent for TSLint\'s "tslint-editor-setting-two"',
161161
);
162162
});
163163
});

src/reporting/reportEditorSettingConversionResults.ts

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -28,7 +28,8 @@ export const reportEditorSettingConversionResults = (
2828
if (editorSettingConversionResults.missing.length !== 0) {
2929
const missingEditorSettingOutputMapping = (
3030
editorSetting: Pick<EditorSetting, "editorSettingName">,
31-
) => `${editorSetting.editorSettingName} does not yet have an ESLint equivalent.${EOL}`;
31+
) =>
32+
`tslint-to-eslint-config does not know the ESLint equivalent for TSLint's "${editorSetting.editorSettingName}"${EOL}`;
3233
logMissingConversionTarget(
3334
"editor setting",
3435
missingEditorSettingOutputMapping,

0 commit comments

Comments
 (0)