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

Enable typescript for test files and fix sourcemaps #392

Merged
merged 2 commits into from
Sep 7, 2021

Conversation

pcardune
Copy link
Collaborator

@pcardune pcardune commented Sep 7, 2021

Turns out there is a bug in karma-webpack v5 that made sourcemaps stop working. I found the workaround in this issue: codymikol/karma-webpack#493.

I think in theory this might make running the tests a bit slower, but I didn't notice any difference in performance when I tried it out locally.

@pcardune pcardune requested a review from schanzer September 7, 2021 16:59
@schanzer
Copy link
Member

schanzer commented Sep 7, 2021

Whoa! MY HERO

@schanzer schanzer merged commit 319833e into master Sep 7, 2021
@pcardune pcardune deleted the pcardune-tests-i391 branch September 7, 2021 17:30
@pcardune pcardune linked an issue Sep 7, 2021 that may be closed by this pull request
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Investigate Heisenbugs
2 participants