-
-
Notifications
You must be signed in to change notification settings - Fork 3.5k
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
Update compile test to use ui_test 0.23 #13245
Conversation
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.
Tested locally and it looks good, thanks for the quick work! I only have one nit about a confusing line of code (I'm assuming it is the replacement for dependencies_crate_manifest_path
).
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.
ok for me with the comment on config.comment_defaults
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.
Please fix CI, and you're good to go!
With bevyengine#13245 merged ui tests no longer specify dependency versions. Revert the change to `release.yml` made in bevyengine#12810 as it's no longer required.
Objective
Closes #13241
Solution
Update test utils to use
ui_test
0.23.0.Testing
cc @BD103