-
Notifications
You must be signed in to change notification settings - Fork 163
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
empty commit for CI check #443
base: main
Are you sure you want to change the base?
Conversation
Well the cloudflare_zlib test failed but not in the expected way... |
Thanks a lot! That's true! The roundtrip test fails which is very strange. Compilation is working though, so that's probably a good sign. What's not great is that CI seems to be broken by Rust v1.84 as they don't ship 'wasm32-wasi' anymore, but now provide 'wasm32-wasip1'? Would you be able to swap this out (it will work, I did the same already elsewhere)? Further, could you try to pin the |
Could you try to push a commit with #441 reverted to see if that's related to the roundtrip test? It probably is not going to affect it. It's notable that the roundtrip test isn't failing on MacOS when running it locally.
|
This reverts commit 8d1467d.
Tried reverting - it still builds fine but it's failing on nightly instead of macos now, and on one of the stable tests... Looking at the roundtrip tests, one thing I noticed that is not entirely ideal, though I don't know if it has any bearing on the results is that it is using data that is generated using a random function so it won't be consistent between runs |
Thanks for trying! I will try to tackle this today and fix it one way or another - CI has to be back. |
No description provided.