From c070037357aff2e97429234cf11fe866ad13341f Mon Sep 17 00:00:00 2001 From: Igor Minar Date: Thu, 30 Jan 2020 14:33:04 -0800 Subject: [PATCH] refactor(compiler): rename diagnostics/src/code.ts to diagnostics/src/error_code.ts (#35067) the new filename is less ambiguous and better reflects the name of the symbol defined in it. PR Close #35067 --- packages/compiler-cli/src/ngtsc/diagnostics/index.ts | 2 +- packages/compiler-cli/src/ngtsc/diagnostics/src/error.ts | 2 +- .../src/ngtsc/diagnostics/src/{code.ts => error_code.ts} | 6 ++++++ tools/public_api_guard/BUILD.bazel | 2 +- 4 files changed, 9 insertions(+), 3 deletions(-) rename packages/compiler-cli/src/ngtsc/diagnostics/src/{code.ts => error_code.ts} (98%) diff --git a/packages/compiler-cli/src/ngtsc/diagnostics/index.ts b/packages/compiler-cli/src/ngtsc/diagnostics/index.ts index 1f1cabc93d..660c12a1d0 100644 --- a/packages/compiler-cli/src/ngtsc/diagnostics/index.ts +++ b/packages/compiler-cli/src/ngtsc/diagnostics/index.ts @@ -6,6 +6,6 @@ * found in the LICENSE file at https://angular.io/license */ -export {ErrorCode, ngErrorCode} from './src/code'; export {FatalDiagnosticError, isFatalDiagnosticError, makeDiagnostic} from './src/error'; +export {ErrorCode, ngErrorCode} from './src/error_code'; export {replaceTsWithNgInErrors} from './src/util'; diff --git a/packages/compiler-cli/src/ngtsc/diagnostics/src/error.ts b/packages/compiler-cli/src/ngtsc/diagnostics/src/error.ts index 59c0886f38..2454c86418 100644 --- a/packages/compiler-cli/src/ngtsc/diagnostics/src/error.ts +++ b/packages/compiler-cli/src/ngtsc/diagnostics/src/error.ts @@ -8,7 +8,7 @@ import * as ts from 'typescript'; -import {ErrorCode} from './code'; +import {ErrorCode} from './error_code'; export class FatalDiagnosticError { constructor(readonly code: ErrorCode, readonly node: ts.Node, readonly message: string) {} diff --git a/packages/compiler-cli/src/ngtsc/diagnostics/src/code.ts b/packages/compiler-cli/src/ngtsc/diagnostics/src/error_code.ts similarity index 98% rename from packages/compiler-cli/src/ngtsc/diagnostics/src/code.ts rename to packages/compiler-cli/src/ngtsc/diagnostics/src/error_code.ts index 868ab67317..b306ecc00b 100644 --- a/packages/compiler-cli/src/ngtsc/diagnostics/src/code.ts +++ b/packages/compiler-cli/src/ngtsc/diagnostics/src/error_code.ts @@ -6,6 +6,9 @@ * found in the LICENSE file at https://angular.io/license */ +/** + * @publicApi + */ export enum ErrorCode { DECORATOR_ARG_NOT_LITERAL = 1001, DECORATOR_ARITY_WRONG = 1002, @@ -125,6 +128,9 @@ export enum ErrorCode { INJECTABLE_DUPLICATE_PROV = 9001, } +/** + * @internal + */ export function ngErrorCode(code: ErrorCode): number { return parseInt('-99' + code); } diff --git a/tools/public_api_guard/BUILD.bazel b/tools/public_api_guard/BUILD.bazel index f2ee5ea534..922324ebc7 100644 --- a/tools/public_api_guard/BUILD.bazel +++ b/tools/public_api_guard/BUILD.bazel @@ -69,7 +69,7 @@ ts_api_guardian_test( # explicit target because the d.ts file is nested in the core and not part of typical public d.ts api ts_api_guardian_test( name = "error_code_api", - actual = "angular/packages/compiler-cli/src/ngtsc/diagnostics/src/code.d.ts", + actual = "angular/packages/compiler-cli/src/ngtsc/diagnostics/src/error_code.d.ts", data = [ ":error_code.d.ts", "//packages/compiler-cli/src/ngtsc/diagnostics",