Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

feat(webdriver): support geckodriver for linux-aarch64 #1371

Merged
merged 1 commit into from
Jun 26, 2024

Conversation

LeoBorai
Copy link
Contributor

Make sure these boxes are checked! 📦✅

  • You have the latest version of rustfmt installed
$ rustup component add rustfmt
  • You ran cargo fmt on the code base before submitting
  • [-] You reference which issue is being closed in the PR text

✨✨ 😄 Thanks so much for contributing to wasm-pack! 😄 ✨✨

Introduces support to download Geckodriver in Linux aarch64 based on artifacts available
in Mozilla's releases: https://github.com/mozilla/geckodriver/releases.

@LeoBorai
Copy link
Contributor Author

Interesting, I changed geckodriver's code but chromedriver is failing here 🤔 https://github.com/rustwasm/wasm-pack/actions/runs/8069923360/job/22046060320?pr=1371#step:7:472

@LeoBorai
Copy link
Contributor Author

@LeoBorai
Copy link
Contributor Author

LeoBorai commented Feb 27, 2024

The version fetched from https://googlechromelabs.github.io/chrome-for-testing/last-known-good-versions.json seems to be cached in tests environments, due to the fact that fetching such URL returns: "channel":"Stable","version":"122.0.6261.94","revision":"1250580" while both tests in different PRs point to: chromedriver version: 122.0.6261.69

@drager
Copy link
Member

drager commented Jun 25, 2024

I just merged #1391. Can you rebase master into this branch? :)

@LeoBorai
Copy link
Contributor Author

I just merged #1391. Can you rebase master into this branch? :)

Hi! Sure!

@LeoBorai LeoBorai force-pushed the feat/geckodriver-linux-aarch64 branch from 69dad53 to e885bc6 Compare June 26, 2024 01:13
@LeoBorai
Copy link
Contributor Author

Hi @drager! I think this is good to go now!

Copy link
Member

@drager drager 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!

@drager drager merged commit bd94075 into rustwasm:master Jun 26, 2024
5 checks passed
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants