Skip to content

cargo update #294

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

Open
wants to merge 4 commits into
base: main
Choose a base branch
from
Open

cargo update #294

wants to merge 4 commits into from

Conversation

jonhoo
Copy link
Owner

@jonhoo jonhoo commented Mar 22, 2025

No description provided.

jonhoo added 2 commits March 22, 2025 15:43
@jonhoo
Copy link
Owner Author

jonhoo commented Mar 22, 2025

This currently fails because the webdriver crate failed to make the capabilities field of NewSessionParameters pub when converting it from an enum to a struct as part of the removal of legacy support in 0.51. I've reported it to the mailing list, but we'll be blocked from upgrading until a fix lands.

@jonhoo
Copy link
Owner Author

jonhoo commented Mar 22, 2025

The email doesn't appear to have gone through, so I've commented on the relevant upstream change.

@jonhoo
Copy link
Owner Author

jonhoo commented Mar 24, 2025

Upstream bug report: https://bugzilla.mozilla.org/show_bug.cgi?id=1955903

@jonhoo
Copy link
Owner Author

jonhoo commented Mar 29, 2025

Worth noting that this will be a breaking change, but that's okay since we'll already have to break with #289 and #285.

@jonhoo
Copy link
Owner Author

jonhoo commented Mar 29, 2025

Well this is awkward: the current versions of Firefox and Chrome both only allow integer values for the x and y fields of pointer action move commands, but webdriver now encodes them as f64. I have followed up upstream in the same bug report.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant