-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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
Bump accesskit to 0.18 and make it a workspace dependency #5783
base: master
Are you sure you want to change the base?
Conversation
Preview available at https://egui-pr-preview.github.io/pr/5783-bump-accesskit |
ee4f632
to
cee1ad9
Compare
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.
Thank you!
Cargo.toml
Outdated
@@ -83,7 +85,7 @@ glutin = { version = "0.32.0", default-features = false } | |||
glutin-winit = { version = "0.5.0", default-features = false } | |||
home = "0.5.9" | |||
image = { version = "0.25", default-features = false } | |||
kittest = { version = "0.1" } | |||
kittest = { git = "https://github.com/valadaptive/kittest", rev = "9b2b8cd" } |
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.
For now we can use kittest from master. I'll release a new version with the next egui release.
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.
Updated kittest to the latest git version. cargo-deny is complaining about a wildcard dependency; not sure why. Do I need to specify an explicit git revision?
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.
Hmm, no idea 🤔 Maybe it works when added as a patch instead?
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.
Alright, it seems that cargo-deny doesn't allow git dependencies unless the allow-wildcard-paths
option is enabled, and doesn't allow git dependencies in public crates regardless of what that option is set to. Since egui_kittest is a public crate, cargo-deny will fail until a new version of kittest is published that we can switch to.
54e4815
to
75a6b52
Compare
Specifying the kittest version seems to do the trick |
This can't be merged until kittest's accesskit is bumped
and a new version of rfd is released.