Skip to content

Conversation

crisbeto
Copy link
Member

Adds some suggestions for potential pipes to be imported when raising the missing pipe diagnostic.

Fixes #62136.

Adds some suggestions for potential pipes to be imported when raising the missing pipe diagnostic.

Fixes angular#62136.
@crisbeto crisbeto requested a review from AndrewKushnir June 19, 2025 11:56
@crisbeto crisbeto added action: review The PR is still awaiting reviews from at least one requested reviewer target: patch This PR is targeted for the next patch release labels Jun 19, 2025
@angular-robot angular-robot bot added the area: compiler Issues related to `ngc`, Angular's template compiler label Jun 19, 2025
@ngbot ngbot bot added this to the Backlog milestone Jun 19, 2025

if (isStandalone) {
errorMsg +=
`\nTo fix this, import the "${suggestedClassName}" class from "${suggestedImport}"` +
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I was pondering whether to also say something like "...or import it from your own codebase" here.

Copy link
Contributor

@AndrewKushnir AndrewKushnir left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks great, thanks @crisbeto 👍

@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 Jun 19, 2025
@crisbeto
Copy link
Member Author

This PR was merged into the repository by commit ed5062a.

The changes were merged into the following branches: main, 20.0.x

@crisbeto crisbeto closed this in ed5062a Jun 20, 2025
crisbeto added a commit that referenced this pull request Jun 20, 2025
Adds some suggestions for potential pipes to be imported when raising the missing pipe diagnostic.

Fixes #62136.

PR Close #62146
@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 Aug 1, 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: compiler Issues related to `ngc`, Angular's template compiler target: patch This PR is targeted for the next patch release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Make "No pipe found" error message more actionable for common pipes
2 participants