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

reuse compilation result of ts-loader #107

Merged
merged 14 commits into from
Jul 10, 2024
Merged

Conversation

jantimon
Copy link
Owner

@jantimon jantimon commented Jun 30, 2024

right now the special import e.g. ./page.yak.module.css!=!./page?./page.yak.module.css is executed by the css-loader directly from it's original source

this means that the typescript transformation has to be run again for the entire code although it was just transformed by the ts-loader

to workaround this issue the compilation result of ./page could be reused for ./page.yak.module.css!=!./page?./page.yak.module.css

such a workaround could also help us for swc as executing swc from the css-loader with the very same options might be tricky

Copy link

vercel bot commented Jun 30, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
next-yak-benchmark ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jul 10, 2024 9:00am
yacijs ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jul 10, 2024 9:00am

Copy link

codspeed-hq bot commented Jun 30, 2024

CodSpeed Performance Report

Merging #107 will improve performances by 12.1%

Comparing feature/faster-compilation (231dd5b) with main (ddc1b3d)

Summary

⚡ 2 improvements

Benchmarks breakdown

Benchmark main feature/faster-compilation Change
render KanjiLetterComponentStyled 322.1 ms 287.3 ms +12.1%
render KanjiLetterComponentYak 239.5 ms 148.3 ms +61.51%

@jantimon
Copy link
Owner Author

jantimon commented Jul 9, 2024

I am no longer able to build the playground locally 🤷‍♂️

@Mad-Kat could you please take a look?

@jantimon jantimon requested a review from Mad-Kat July 9, 2024 06:25
Mad-Kat
Mad-Kat previously approved these changes Jul 10, 2024
package.json Show resolved Hide resolved
Mad-Kat
Mad-Kat previously approved these changes Jul 10, 2024
@jantimon jantimon merged commit 0fc8271 into main Jul 10, 2024
7 checks passed
@jantimon jantimon deleted the feature/faster-compilation branch July 10, 2024 10:19
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.

2 participants