-
Notifications
You must be signed in to change notification settings - Fork 25
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
Support proc macro test dependencies #204
Conversation
Should be able to test this by adding serde_derive and using it in a |
7888276
to
216298e
Compare
Done. Also fixed the formatting and added two commits to remove the diffs caused by automatic blessing of all tests. |
216298e
to
c08ec5f
Compare
Rebased |
This avoids test diffs when RUST_BACKTRACE=1 is set in in the shell config.
c08ec5f
to
4ca90a8
Compare
Removed an unnecessary change. I don't understand why the last CI run only failed on Windows: https://github.com/oli-obk/ui_test/actions/runs/8216901979/job/22472040330 |
I think that may be an orthogonal bug that just didn't surface before. Ralf reported it in miri recently |
It was two bugs! 😆 works locally on windows now. Let's see what CI says |
ah well, now I broke unixes 💀 |
e5e33f1
to
dda7650
Compare
Fixes #203