-
-
Notifications
You must be signed in to change notification settings - Fork 8.1k
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
Add rexml dependency #9007
Add rexml dependency #9007
Conversation
Since ruby 3.0.0, rexml is a bundled gem. So it must be depend explicitly.
Kudos, SonarCloud Quality Gate passed! |
Since Selenium 4 has removed the Firefox extension, I think the right answer for trunk is to delete the requires from firefox.rb rather than adding the dependency. I haven't had a chance to pull Ruby 3 yet to check it out, yet... |
There are also other places with REXML: selenium/rb/lib/selenium/server.rb Lines 112 to 121 in 109c25a
|
Also have this problem, fixed it by adding |
@ioquatix go ahead and use the beta. Very little will change in Ruby bindings between now and final release. |
Description
Since ruby 3.0.0, rexml is a bundled gem.
So it must be depend explicitly.
Motivation and Context
https://stackoverflow.com/questions/65479863/rails-6-1-what-is-preventing-tests-from-running/65480744#65480744
Types of changes
Checklist