Skip to content

Conversation

@crisbeto
Copy link
Member

@crisbeto crisbeto commented Dec 11, 2025

After the recent infra changes we should use relative imports within the same package (e.g. one aria symbol importing another one), whereas absolute imports should only be between packages (e.g. aria importing from cdk).

These changes update all the imports and add private re-exports for DeferredContent. This is necessary to work around a compiler issue.

@crisbeto crisbeto added the target: patch This PR is targeted for the next patch release label Dec 11, 2025
@pullapprove pullapprove bot requested review from mmalerba and ok7sai December 11, 2025 09:02
After the recent infra changes we should use relative imports within the same package (e.g. one `aria` symbol importing another one), whereas absolute imports should only be between packages (e.g. `aria` importing from `cdk`).

These changes update all the imports and add private re-exports for `DeferredContent`. This is necessary to work around a compiler issue.
@crisbeto crisbeto force-pushed the aria-private-imports branch from 244daf8 to 52a566f Compare December 11, 2025 09:09
@crisbeto crisbeto removed the request for review from mmalerba December 11, 2025 09:20
@crisbeto crisbeto added the action: merge The PR is ready for merge by the caretaker label Dec 11, 2025
@crisbeto crisbeto merged commit 5d98011 into angular:main Dec 11, 2025
23 of 26 checks passed
@crisbeto
Copy link
Member Author

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

crisbeto added a commit that referenced this pull request Dec 11, 2025
After the recent infra changes we should use relative imports within the same package (e.g. one `aria` symbol importing another one), whereas absolute imports should only be between packages (e.g. `aria` importing from `cdk`).

These changes update all the imports and add private re-exports for `DeferredContent`. This is necessary to work around a compiler issue.

(cherry picked from commit 5d98011)
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 target: patch This PR is targeted for the next patch release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants