From f77bd0a2bd0650fe67cbe071ff56a7d2bf9dc1b9 Mon Sep 17 00:00:00 2001 From: Paul Gschwendtner Date: Tue, 20 Jun 2023 13:58:05 +0000 Subject: [PATCH] refactor: fix lint warning in compiler code by adding explicit `override` (#50772) We have a lint rule configured that enforces that any abstract member implementation uses an explicit `override` identifier. This ensures that downstream classes will have errors if the parent abstract class suddenly removes the abstract member. The lint rule, living in the dev-infra repository, occasionally does miss some places due to a temporary TS version mismatch that causes syntax kind indices to be different. Looks like we are now matching again and there is a new lint failure that got introduced recently. This commit fixes that error. PR Close #50772 --- packages/compiler/src/output/output_ast.ts | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/packages/compiler/src/output/output_ast.ts b/packages/compiler/src/output/output_ast.ts index 7d1915f9b12..8c79fcd1ea2 100644 --- a/packages/compiler/src/output/output_ast.ts +++ b/packages/compiler/src/output/output_ast.ts @@ -988,7 +988,7 @@ export class LiteralArrayExpr extends Expression { return visitor.visitLiteralArrayExpr(this, context); } - clone(): LiteralArrayExpr { + override clone(): LiteralArrayExpr { return new LiteralArrayExpr(this.entries.map(e => e.clone()), this.type, this.sourceSpan); } }