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

Support proc macro test dependencies #204

Merged
merged 5 commits into from
Mar 20, 2024

Conversation

bjorn3
Copy link
Contributor

@bjorn3 bjorn3 commented Feb 24, 2024

Fixes #203

@oli-obk
Copy link
Owner

oli-obk commented Feb 24, 2024

Should be able to test this by adding serde_derive and using it in a //@run test

@bjorn3 bjorn3 force-pushed the fix_proc_macro_deps branch from 7888276 to 216298e Compare February 24, 2024 19:24
@bjorn3
Copy link
Contributor Author

bjorn3 commented Feb 24, 2024

Done. Also fixed the formatting and added two commits to remove the diffs caused by automatic blessing of all tests.

@bjorn3 bjorn3 force-pushed the fix_proc_macro_deps branch from 216298e to c08ec5f Compare March 9, 2024 20:07
@bjorn3
Copy link
Contributor Author

bjorn3 commented Mar 9, 2024

Rebased

bjorn3 added 2 commits March 10, 2024 09:10
This avoids test diffs when RUST_BACKTRACE=1 is set in in the shell
config.
@bjorn3 bjorn3 force-pushed the fix_proc_macro_deps branch from c08ec5f to 4ca90a8 Compare March 10, 2024 09:22
@bjorn3
Copy link
Contributor Author

bjorn3 commented Mar 10, 2024

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

@oli-obk
Copy link
Owner

oli-obk commented Mar 10, 2024

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

@oli-obk
Copy link
Owner

oli-obk commented Mar 13, 2024

It was two bugs! 😆 works locally on windows now. Let's see what CI says

@oli-obk
Copy link
Owner

oli-obk commented Mar 13, 2024

ah well, now I broke unixes 💀

@oli-obk oli-obk force-pushed the fix_proc_macro_deps branch from e5e33f1 to dda7650 Compare March 20, 2024 10:48
@oli-obk oli-obk merged commit b874810 into oli-obk:main Mar 20, 2024
5 checks passed
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.

Support proc macro test dependencies
2 participants