Skip to content

[Cherry pick] build: reuse fake core in type checking tests (#54344)#54398

Closed
crisbeto wants to merge 1 commit intoangular:17.2.xfrom
crisbeto:17.2.x-typecheck-tests
Closed

[Cherry pick] build: reuse fake core in type checking tests (#54344)#54398
crisbeto wants to merge 1 commit intoangular:17.2.xfrom
crisbeto:17.2.x-typecheck-tests

Conversation

@crisbeto
Copy link
Copy Markdown
Member

Cherry picks #54344 into the RC branch.

Currently we have two fake copies of @angular/core in the compiler tests which can be out of sync and cause inconsistent tests. These changes reuse a single copy instead.

Currently we have two fake copies of `@angular/core` in the compiler tests which can be out of sync and cause inconsistent tests. These changes reuse a single copy instead.

PR Close angular#54344
@crisbeto crisbeto added the target: rc This PR is targeted for the next release-candidate label Feb 12, 2024
@pullapprove pullapprove bot requested a review from atscott February 12, 2024 20:01
@angular-robot angular-robot bot added the area: build & ci Related the build and CI infrastructure of the project label Feb 12, 2024
@ngbot ngbot bot added this to the Backlog milestone Feb 12, 2024
@crisbeto
Copy link
Copy Markdown
Member Author

Disabling PullApprove since it's just a cherry-pick.

@AndrewKushnir AndrewKushnir added the action: merge The PR is ready for merge by the caretaker label Feb 12, 2024
@AndrewKushnir
Copy link
Copy Markdown
Contributor

This PR was merged into the repository by commit 277540c.

AndrewKushnir pushed a commit that referenced this pull request Feb 12, 2024
Currently we have two fake copies of `@angular/core` in the compiler tests which can be out of sync and cause inconsistent tests. These changes reuse a single copy instead.

PR Close #54344

PR Close #54398
@angular-automatic-lock-bot
Copy link
Copy Markdown

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 Mar 14, 2024
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 PullApprove: disable target: rc This PR is targeted for the next release-candidate

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants