Skip to content

Comments

[TS] Use 'override' when overriding class items#3852

Merged
coolreader18 merged 2 commits intomasterfrom
noa/ts-override
Dec 9, 2025
Merged

[TS] Use 'override' when overriding class items#3852
coolreader18 merged 2 commits intomasterfrom
noa/ts-override

Conversation

@coolreader18
Copy link
Collaborator

Description of Changes

This should fix part of #3503. Adds an override modifier to generated code and fixes a warning from the angular compiler.

Expected complexity level and risk

1

Testing

  • Code works fine with the added override modifier.
  • Perhaps we should have an angular test project?

@coolreader18 coolreader18 added this pull request to the merge queue Dec 9, 2025
Merged via the queue into master with commit bea7a7d Dec 9, 2025
27 checks passed
@coolreader18 coolreader18 deleted the noa/ts-override branch December 9, 2025 21:48
@joshua-spacetime joshua-spacetime mentioned this pull request Dec 11, 2025
2 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants