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

Bump accesskit to 0.18 and make it a workspace dependency #5783

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

valadaptive
Copy link
Contributor

@valadaptive valadaptive commented Mar 12, 2025

Copy link

Preview available at https://egui-pr-preview.github.io/pr/5783-bump-accesskit
Note that it might take a couple seconds for the update to show up after the preview_build workflow has completed.

@lucasmerlin lucasmerlin added the dependencies Pull requests that update a dependency file label Mar 13, 2025
Copy link
Collaborator

@lucasmerlin lucasmerlin left a 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" }
Copy link
Collaborator

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.

Copy link
Contributor Author

@valadaptive valadaptive Mar 13, 2025

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?

Copy link
Collaborator

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?

Copy link
Contributor Author

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.

@lucasmerlin
Copy link
Collaborator

Specifying the kittest version seems to do the trick

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants