Remove 'external module' wording from rules (#340)

The term now is just 'module'. I improved the wording while I was here.
This commit is contained in:
Nathan Shively-Sanders 2021-08-11 17:18:25 -07:00 коммит произвёл GitHub
Родитель fc56160f99
Коммит cd084e87da
Не найден ключ, соответствующий данной подписи
Идентификатор ключа GPG: 4AEE18F83AFDEB23
3 изменённых файлов: 4 добавлений и 4 удалений

Просмотреть файл

@ -6,7 +6,7 @@ import { failure, getCommonDirectoryName } from "../util";
export class Rule extends Lint.Rules.TypedRule {
static metadata: Lint.IRuleMetadata = {
ruleName: "no-declare-current-package",
description: "Don't use an ambient module declaration of the current package; use an external module.",
description: "Don't use an ambient module declaration of the current package; use a normal module.",
optionsDescription: "Not configurable.",
options: null,
type: "functionality",

Просмотреть файл

@ -6,7 +6,7 @@ import { failure } from "../util";
export class Rule extends Lint.Rules.AbstractRule {
static metadata: Lint.IRuleMetadata = {
ruleName: "no-single-declare-module",
description: "Don't use an ambient module declaration if you can use an external module file.",
description: "Don't use an ambient module declaration if there's just one -- write it as a normal module.",
rationale: "Cuts down on nesting",
optionsDescription: "Not configurable.",
options: null,
@ -16,7 +16,7 @@ export class Rule extends Lint.Rules.AbstractRule {
static FAILURE_STRING = failure(
Rule.metadata.ruleName,
"File has only 1 module declaration — write it as an external module.");
"File has only 1 ambient module declaration. Move the contents outside the ambient module block, rename the file to match the ambient module name, and remove the block.");
apply(sourceFile: ts.SourceFile): Lint.RuleFailure[] {
return this.applyWithFunction(sourceFile, walk);

Просмотреть файл

@ -1,5 +1,5 @@
declare module "foo" {}
~~~~~~~~~~~~~~~~~~~~~~~ [File has only 1 module declaration — write it as an external module. See: https://github.com/Microsoft/dtslint/blob/master/docs/no-single-declare-module.md]
~~~~~~~~~~~~~~~~~~~~~~~ [File has only 1 ambient module declaration. Move the contents outside the ambient module block, rename the file to match the ambient module name, and remove the block. See: https://github.com/Microsoft/dtslint/blob/master/docs/no-single-declare-module.md]
// Other global declarations don't affect this. They should go in "declare global".
interface I {}