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

Fix Ci failing over dead code in tests #12623

Merged
merged 1 commit into from Mar 21, 2024

Conversation

Brezak
Copy link
Contributor

@Brezak Brezak commented Mar 21, 2024

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.

@Brezak
Copy link
Contributor Author

Brezak commented Mar 21, 2024

Now we wait and see if that was all the dead code the compiler recognizes.

@Brezak Brezak changed the title Fix Ci failing over dead code in test Fix Ci failing over dead code in tests Mar 21, 2024
@alice-i-cecile alice-i-cecile added A-Build-System Related to build systems or continuous integration C-Code-Quality A section of code that is hard to understand or change P-Critical This must be fixed immediately or contributors or users will be severely impacted labels Mar 21, 2024
@Brezak
Copy link
Contributor Author

Brezak commented Mar 21, 2024

Should work now. I'll investigate a better solution when CI works again.

@alice-i-cecile alice-i-cecile marked this pull request as ready for review March 21, 2024 16:17
@alice-i-cecile alice-i-cecile added the S-Ready-For-Final-Review This PR has been approved by the community. It's ready for a maintainer to consider merging it label Mar 21, 2024
@alice-i-cecile
Copy link
Member

Looks like we need to bless the new output of the compile-fail tests too.

@alice-i-cecile
Copy link
Member

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.

@Brezak Brezak force-pushed the fix-ci-dead-code@origin branch 3 times, most recently from 1898b59 to 9fc4dc1 Compare March 21, 2024 17:25
@Brezak
Copy link
Contributor Author

Brezak commented Mar 21, 2024

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.

@Brezak
Copy link
Contributor Author

Brezak commented Mar 21, 2024

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.

@Brezak
Copy link
Contributor Author

Brezak commented Mar 21, 2024

It works!!!. I'm sorry it took so long.

@mockersf mockersf added this pull request to the merge queue Mar 21, 2024
@alice-i-cecile
Copy link
Member

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.

@mockersf
Copy link
Member

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.

Yup it's #8962, somehow the output differs depending on your OS...

Merged via the queue into bevyengine:main with commit 69e78bd Mar 21, 2024
26 checks passed
@Brezak Brezak deleted the fix-ci-dead-code@origin branch April 24, 2024 10:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-Build-System Related to build systems or continuous integration C-Code-Quality A section of code that is hard to understand or change P-Critical This must be fixed immediately or contributors or users will be severely impacted S-Ready-For-Final-Review This PR has been approved by the community. It's ready for a maintainer to consider merging it
Projects
None yet
Development

Successfully merging this pull request may close these issues.

CI is failing on Rust 1.77
4 participants