-
Notifications
You must be signed in to change notification settings - Fork 13
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 gradle 8.7 compatibility via org.ysb33r.gradle:grolifant-herd:4.0.0 #44
base: master
Are you sure you want to change the base?
add gradle 8.7 compatibility via org.ysb33r.gradle:grolifant-herd:4.0.0 #44
Conversation
Hey, @jamesfredley! Neither I nor the Geb project maintain this repository. For our part, we're actively working to move from this to Selenium Manager, which solves the same core problem. You can see the in-progress pull request for this at apache/groovy-geb#200. Even without that update to Geb, you can start using Selenium Manager by upgrading to Selenium 4.11 or later. It works just fine with the latest Geb release. I would recommend upgrading to the latest version of Selenium you can in your environment, as Selenium Manager did have a few bugs in the earlier iterations that are now resolved. That may entail Java updates, though, depending on your project. Obviously, you can always use an artifact built from your fork here in the interim, and if @erdi wants to merge this here, more power to you both. :) |
Thanks for the reply. For Grails, we are moving the default Geb setup to use org.testcontainers:selenium via https://github.com/grails/geb/blob/7a50d0aa78671f41ce0237b98b501cf844208081/src/testFixtures/groovy/grails/plugin/geb/ContainerGebSpec.groovy#L49 which is working great. This PR would help users that want to use Gradle 8.6+ with webdriver-binaries-gradle-plugin. |
will resolve:
replaces:
Changes were made to enable tests to run on Windows 11.
Fully tested against: