-
Notifications
You must be signed in to change notification settings - Fork 66
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 breaking due to new dependencies and rustc changes #161
Conversation
a5e77da
to
5c30dd9
Compare
16a3a8a
to
f81c492
Compare
Failing on windows due to an extra function call before main (in Rust) on windows only |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I gave my understanding of what you're doing here -- please correct me if I'm wrong, feel free to commit if not.
out of scope but we/I need to handle that huge block of warnings.
@@ -168,7 +170,7 @@ fn test_backwards_compatibility(target: String, file_name: &str) { | |||
fn normalize_backtrace(input: &str) -> String { | |||
input | |||
.lines() | |||
.take_while(|v| !v.contains("core::panic")) | |||
.take_while(|v| !v.contains("core::panic") && !v.contains("theme_test_helper::main")) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is ignoring a specific kind of panic from theme_test_helper, right?
No description provided.