Skip to content

Conversation

jkrems
Copy link
Contributor

@jkrems jkrems commented Dec 19, 2024

We should only force the type for files that we know are JavaScript. Otherwise we risk breaking the magic type detection done by Karma.

The previous code broke for jasmine.css.

Fixes #29190

We should only force the type for files that we know are JavaScript.
Otherwise we risk breaking the magic type detection done by Karma.

The previous code broke for `jasmine.css`.

Fixes angular#29190
@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 labels Dec 19, 2024
@clydin clydin merged commit 916979c into angular:main Dec 20, 2024
33 checks passed
@clydin
Copy link
Member

clydin commented Dec 20, 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 20, 2025
@jkrems jkrems deleted the jk-jasmine-css 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.

Angular 19.0.6 breaks karma/jasmine on our build server by trying to load jasmine.css as a script module
2 participants