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
Fix Ci failing over dead code in tests #12623
Conversation
Now we wait and see if that was all the dead code the compiler recognizes. |
7a0350c
to
f55bcbf
Compare
Should work now. I'll investigate a better solution when CI works again. |
Looks like we need to bless the new output of the compile-fail tests too. |
f55bcbf
to
d0cef71
Compare
d0cef71
to
5e5a6a7
Compare
Another compile-fail failure in https://github.com/bevyengine/bevy/actions/runs/8378534423/job/22943464592?pr=12623#step:6:261. I've opened #12627 to see how we can make this less brittle. |
1898b59
to
9fc4dc1
Compare
Either I'm doing something wrong or the errors rustc emits look different between Windows and Linux. Blessing the tests locally keeps failing so I've resorted to just copy pasting the expected output from the CI log. |
9fc4dc1
to
3f6b7bb
Compare
If this works now then the issue was that I forgot to add a newline to the end of the missing_deref.fail.stderr file when copying the CI output. |
3f6b7bb
to
43dfc4a
Compare
It works!!!. I'm sorry it took so long. |
No, this was very speedy! Much appreciated: it's a real chore to fix these failures and I love it when contributors tackle them. |
Yup it's #8962, somehow the output differs depending on your OS... |
Objective
Fix Pr CI failing over dead code in tests and main branch CI failing over a missing semicolon. Fixes #12620.
Solution
Add dead_code annotations and a semicolon.