Skip to content

Conversation

mmalerba
Copy link
Contributor

No description provided.

Copy link
Member

@ok7sai ok7sai left a comment

Choose a reason for hiding this comment

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

Since it's a testing util I guess the linter errors are fine?

@mmalerba
Copy link
Contributor Author

yeah, just added some comments to disable those. they're not relevant since the ngOnDestroy method does nothing, I just had to put it there to match the interface of the real API

@mmalerba mmalerba added action: merge The PR is ready for merge by the caretaker target: patch This PR is targeted for the next patch release target: rc This PR is targeted for the next release-candidate and removed target: patch This PR is targeted for the next patch release labels May 14, 2025
@mmalerba mmalerba merged commit 6f73c21 into angular:main May 14, 2025
29 of 31 checks passed
mmalerba added a commit that referenced this pull request May 14, 2025
@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 target: rc This PR is targeted for the next release-candidate
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants