Skip to content

Conversation

@clydin
Copy link
Member

@clydin clydin commented Jan 16, 2026

This commit refactors several Tree implementations (DelegateTree, HostTree, NullTree, ScopedTree) to support the TypeScript isolatedDeclarations compiler option. It uses declaration merging to define the [TreeSymbol] method on an interface and assigns it in the constructor to avoid TS9038 errors regarding computed property names inference.

@clydin clydin added the target: minor This PR is targeted for the next minor release label Jan 16, 2026
@clydin clydin force-pushed the isolated-declarations-5 branch from d20da4b to 8e32605 Compare January 16, 2026 20:52
@clydin clydin force-pushed the isolated-declarations-5 branch from e6e8d56 to 8e32605 Compare January 17, 2026 00:01
@clydin clydin added the action: review The PR is still awaiting reviews from at least one requested reviewer label Jan 17, 2026
@clydin clydin force-pushed the isolated-declarations-5 branch from 8e32605 to d5a4131 Compare January 20, 2026 15:03
…for tree classes

This commit refactors several Tree implementations (`DelegateTree`, `HostTree`, `NullTree`, `ScopedTree`) to support the TypeScript `isolatedDeclarations` compiler option.
It uses declaration merging to define the `[TreeSymbol]` method on an interface and assigns it in the constructor to avoid TS9038 errors regarding computed property names inference.
@clydin clydin force-pushed the isolated-declarations-5 branch from d5a4131 to abc4f57 Compare January 20, 2026 17:31
@clydin clydin added action: merge The PR is ready for merge by the caretaker and removed action: review The PR is still awaiting reviews from at least one requested reviewer labels Jan 20, 2026
@clydin clydin merged commit ecc0797 into angular:main Jan 20, 2026
35 checks passed
@clydin
Copy link
Member Author

clydin commented Jan 20, 2026

This PR was merged into the repository. The changes were merged into the following branches:

@clydin clydin deleted the isolated-declarations-5 branch January 20, 2026 17:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

action: merge The PR is ready for merge by the caretaker area: @angular/cli area: @angular-devkit/schematics target: minor This PR is targeted for the next minor release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants