Skip to content

Conversation

@gadomski
Copy link
Collaborator

@gadomski gadomski commented Dec 8, 2025

Couple of issues:

@gadomski gadomski requested review from alukach and bitner December 8, 2025 11:49
@gadomski gadomski self-assigned this Dec 8, 2025
@kylebarron
Copy link
Member

cargo install will take a little while to build from source; ideally you would cache it but idk how hard that is to do

@gadomski
Copy link
Collaborator Author

gadomski commented Dec 8, 2025

cargo install will take a little while to build from source; ideally you would cache it but idk how hard that is to do

I think the Rust cache action will get it for us, but this repo really shouldn't be that high-velocity so a bit of a extra time doesn't bother me a whole bunch...

Copy link
Member

@alukach alukach left a comment

Choose a reason for hiding this comment

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

Seems reasonable.

Strangely, the cache DOES seem to work for wasm-nodejs but not for wasm-web. No idea why.

Image Image

@gadomski gadomski merged commit 47c0880 into main Dec 8, 2025
27 checks passed
@gadomski gadomski deleted the fix-ci-config branch December 8, 2025 18:37
@github-actions github-actions bot mentioned this pull request Dec 8, 2025
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.

4 participants