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

Fixes downloading on m1 macs from version 106.0.5249.61 #239

Merged
merged 1 commit into from
Sep 29, 2022

Conversation

jmccure
Copy link
Contributor

@jmccure jmccure commented Sep 29, 2022

Fix #237

@carolyncole
Copy link

carolyncole commented Sep 29, 2022

Changed my gemfile to utilize this branch with gem "webdrivers", github: 'jmccure/webdrivers', branch:'fix-m1-downloads' It works for me. Thank you!

@kapoorlakshya
Copy link
Collaborator

@jmccure Thanks for your contribution! Merging to get a release out. Would appreciate a follow up with tests for #apple_filename 👍

@kapoorlakshya kapoorlakshya merged commit 31f10f7 into titusfortner:main Sep 29, 2022
@jmccure
Copy link
Contributor Author

jmccure commented Sep 29, 2022

Thanks @kapoorlakshya , will submit tests also

@kapoorlakshya
Copy link
Collaborator

@jmccure Awesome! v5.2.0 has been released.

@carolyncole Thanks for testing this!

@Mutuba
Copy link

Mutuba commented Sep 30, 2022

late but have tested and it works

facebook-github-bot pushed a commit to facebook/yoga that referenced this pull request Oct 13, 2022
Summary:
This adds the fixtures from #1116 and generates tests.

This adds a good amount of coverage, but I plan to follow up with a diff adding a bit more, e.g. for interactions with flex direction of column when we should no-op, etc. I also discovered the current fixtures do not allow testing shorthand props like "gap" without changes.

This also updates the `webdrivers` gem to respond to a break with chromedriver on m1 macs from 4 days ago titusfortner/webdrivers#239.

Reviewed By: yungsters

Differential Revision: D39922413

fbshipit-source-id: dfc7bda894be8dfcb24e25c19a4df0b09a72ce7e
hank-spokeo pushed a commit to hank-spokeo/webdrivers that referenced this pull request Nov 15, 2022
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.

Webdrivers::NetworkError, probably due to new chromedriver naming
4 participants