Skip to content

Conversation

jkrems
Copy link
Contributor

@jkrems jkrems commented Dec 10, 2024

Ports the logic from WTR and webpack and adds a regression test for this behavior.

Fixes #28813

@jkrems
Copy link
Contributor Author

jkrems commented Dec 11, 2024

Somehow this broke existing behavior. Converting to draft for now while I investigate.

@jkrems jkrems removed the request for review from clydin December 11, 2024 00:20
@jkrems jkrems marked this pull request as draft December 11, 2024 00:20
@jkrems jkrems marked this pull request as ready for review December 11, 2024 21:46
@jkrems
Copy link
Contributor Author

jkrems commented Dec 11, 2024

I'm looking into removing the need for the window.toString hack by fixing this upstream in jasmine, maybe: jasmine/jasmine#2048

@jkrems jkrems requested a review from clydin December 12, 2024 00:08
@jkrems jkrems added the action: review The PR is still awaiting reviews from at least one requested reviewer label Dec 12, 2024
Copy link
Member

@clydin clydin left a comment

Choose a reason for hiding this comment

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

Nice work.
just one typo otherwise LGTM.

@jkrems jkrems added action: merge The PR is ready for merge by the caretaker target: patch This PR is targeted for the next patch release and removed action: review The PR is still awaiting reviews from at least one requested reviewer labels Dec 13, 2024
@jkrems jkrems merged commit a9a3470 into angular:main Dec 13, 2024
32 checks passed
@jkrems
Copy link
Contributor Author

jkrems commented Dec 13, 2024

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

@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 Jan 13, 2025
@jkrems jkrems deleted the jk-jasmine-clock branch August 20, 2025 14:07
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: @angular-devkit/build-angular target: patch This PR is targeted for the next patch release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Karma builder issue with jasmine.clock() with "builderMode: application"
2 participants