Skip to content
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

docs(core): Deprecate ChangeDetectorRef.checkNoChanges #52431

Closed

Conversation

atscott
Copy link
Contributor

@atscott atscott commented Oct 27, 2023

The checkNoChanges method does not belong in the API of production interface. checkNoChanges is limited to testing and should not be used in any application code. Test code should use ComponentFixture instead of ChangeDetectorRef. Additionally, it is not desirable to have the checkNoChanges API available in a context where detectChanges is not run first.

DEPRECATED: ChangeDetectorRef.checkNoChanges is deprecated.

Test code should use ComponentFixture instead of ChangeDetectorRef. Application code should not call ChangeDetectorRef.checkNoChanges directly.

@angular-robot angular-robot bot added the detected: deprecation PR contains a commit with a deprecation label Oct 27, 2023
@atscott atscott changed the title refactor(core): Deprecate ChangeDetectorRef.checkNoChanges docs(core): Deprecate ChangeDetectorRef.checkNoChanges Oct 27, 2023
@atscott atscott added the target: rc This PR is targeted for the next release-candidate label Oct 27, 2023
@atscott atscott force-pushed the deprecateChangeDetectorRefDetectChanges branch from 09593d3 to abd9d56 Compare October 27, 2023 21:19
@ngbot ngbot bot added this to the Backlog milestone Oct 27, 2023
@pkozlowski-opensource
Copy link
Member

cleanup: CI is read (needs yarn bazel run //packages/core:core_api.accept)

@pkozlowski-opensource pkozlowski-opensource added the action: cleanup The PR is in need of cleanup, either due to needing a rebase or in response to comments from reviews label Oct 30, 2023
The `checkNoChanges` method does not belong in the API of production interface. `checkNoChanges` is
limited to testing and should not be used in any application code. Test
code should use `ComponentFixture` instead of `ChangeDetectorRef`.
Additionally, it is not desirable to have the `checkNoChanges` API
available in a context where `detectChanges` is not run first.

DEPRECATED: `ChangeDetectorRef.checkNoChanges` is deprecated.

Test code should use `ComponentFixture` instead of `ChangeDetectorRef`.
Application code should not call `ChangeDetectorRef.checkNoChanges` directly.
Copy link
Contributor

@jessicajaniuk jessicajaniuk left a comment

Choose a reason for hiding this comment

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

reviewed-for: public-api

@atscott atscott added action: merge The PR is ready for merge by the caretaker merge: caretaker note Alert the caretaker performing the merge to check the PR for an out of normal action needed or note and removed action: cleanup The PR is in need of cleanup, either due to needing a rebase or in response to comments from reviews labels Oct 30, 2023
@atscott
Copy link
Contributor Author

atscott commented Oct 30, 2023

caretaker note: docs only, no need for g3 tests

@atscott atscott modified the milestones: Backlog, v17-final Oct 30, 2023
Copy link
Member

@pkozlowski-opensource pkozlowski-opensource left a comment

Choose a reason for hiding this comment

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

LGTM

Reviewed-for: public-api
Reviewed-for: fw-core

@alxhub
Copy link
Member

alxhub commented Oct 31, 2023

This PR was merged into the repository by commit 75725fb.

@alxhub alxhub closed this in 75725fb Oct 31, 2023
alxhub pushed a commit that referenced this pull request Oct 31, 2023
The `checkNoChanges` method does not belong in the API of production interface. `checkNoChanges` is
limited to testing and should not be used in any application code. Test
code should use `ComponentFixture` instead of `ChangeDetectorRef`.
Additionally, it is not desirable to have the `checkNoChanges` API
available in a context where `detectChanges` is not run first.

DEPRECATED: `ChangeDetectorRef.checkNoChanges` is deprecated.

Test code should use `ComponentFixture` instead of `ChangeDetectorRef`.
Application code should not call `ChangeDetectorRef.checkNoChanges` directly.

PR Close #52431
@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 Dec 1, 2023
tbondwilkinson pushed a commit to tbondwilkinson/angular that referenced this pull request Dec 6, 2023
The `checkNoChanges` method does not belong in the API of production interface. `checkNoChanges` is
limited to testing and should not be used in any application code. Test
code should use `ComponentFixture` instead of `ChangeDetectorRef`.
Additionally, it is not desirable to have the `checkNoChanges` API
available in a context where `detectChanges` is not run first.

DEPRECATED: `ChangeDetectorRef.checkNoChanges` is deprecated.

Test code should use `ComponentFixture` instead of `ChangeDetectorRef`.
Application code should not call `ChangeDetectorRef.checkNoChanges` directly.

PR Close angular#52431
ChellappanRajan pushed a commit to ChellappanRajan/angular that referenced this pull request Jan 23, 2024
The `checkNoChanges` method does not belong in the API of production interface. `checkNoChanges` is
limited to testing and should not be used in any application code. Test
code should use `ComponentFixture` instead of `ChangeDetectorRef`.
Additionally, it is not desirable to have the `checkNoChanges` API
available in a context where `detectChanges` is not run first.

DEPRECATED: `ChangeDetectorRef.checkNoChanges` is deprecated.

Test code should use `ComponentFixture` instead of `ChangeDetectorRef`.
Application code should not call `ChangeDetectorRef.checkNoChanges` directly.

PR Close angular#52431
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 detected: deprecation PR contains a commit with a deprecation merge: caretaker note Alert the caretaker performing the merge to check the PR for an out of normal action needed or note target: rc This PR is targeted for the next release-candidate
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants