8000 Selectorless follow ups by crisbeto · Pull Request #61307 · angular/angular · GitHub
[go: up one dir, main page]

Skip to content

Selectorless follow ups #61307

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
wants to merge 3 commits into from
Closed

Conversation

crisbeto
Copy link
Member

Includes a few follow-up changes for selectorless:

refactor(compiler): indicate in AST if node is self-closing

Follow-up from #61240 (comment). Adds a isSelfClosing property on element-like AST nodes so consumers can easily determine if it's self-closing, rather than having to look at the spans. This is useful for migrations and in the language service.

build: set up runtime tests for selectorless

Sets up the tests for the selectorless runtime so that we can easily start writing them when we get to it. The tests need to be AoT compiled so they're defined as a separate target from the other acceptance tests.

refactor(language-service): rename all references to selectorless directives

Follow-up to #61240 that adds renaming support for selectorless components/directives both from the template and from the TypeScript source.

crisbeto added 3 commits May 13, 2025 11:14
Follow-up from angular#61240 (comment). Adds a `isSelfClosing` property on element-like AST nodes so consumers can easily determine if it's self-closing, rather than having to look at the spans. This is useful for migrations and in the language service.
Sets up the tests for the selectorless runtime so that we can easily start writing them when we get to it. The tests need to be AoT compiled so they're defined as a separate target from the other `acceptance` tests.
…ectives

Follow-up to angular#61240 that adds renaming support for selectorless components/directives both from the template and from the TypeScript source.
@crisbeto crisbeto added action: review The PR is still awaiting reviews from at least one requested reviewer target: minor This PR is targeted for the next minor release labels May 13, 2025
@angular-robot angular-robot bot added area: build & ci Related the build and CI infrastructure of the project area: compiler Issues related to `ngc`, Angular's template compiler area: language-service Issues related to Angular's VS Code language service labels May 13, 2025
@ngbot ngbot bot added this to the Backlog milestone May 13, 2025
@crisbeto crisbeto requested a review from devversion May 13, 2025 13:39
@crisbeto crisbeto marked this pull request as ready for review May 13, 2025 13:39
@crisbeto crisbeto 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 May 13, 2025
@crisbeto
Copy link
Member Author

This PR was merged into the repository by commit 3aef3e6.

The changes were merged into the following branches: main

@crisbeto crisbeto closed this in eae1083 May 14, 2025
crisbeto added a commit that referenced this pull request May 14, 2025
Sets up the tests for the selectorless runtime so that we can easily start writing them when we get to it. The tests need to be AoT compiled so they're defined as a separate target from the other `acceptance` tests.

PR Close #61307
crisbeto added a commit that referenced this pull request May 14, 2025
…ectives (#61307)

Follow-up to #61240 that adds renaming support for selectorless components/directives both from the template and from the TypeScript source.

PR Close #61307
@angular-automatic-lock-bot
Copy link

This issue has been automatically locked due to inactivity.
Please file a new issue if you are encountering a similar or related problem.

Read more about our automatic conversation locking policy.

This action has been performed automatically by a bot.

@angular-automatic-lock-bot angular-automatic-lock-bot bot locked and limited conversation to collaborators Jun 14, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
action: merge The PR is ready for merge by the caretaker area: build & ci Related the build and CI infrastructure of the project area: compiler Issues related to `ngc`, Angular's template compiler area: language-service Issues related to Angular's VS Code language service 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
0