Skip to content

Conversation

@titusfortner
Copy link
Member

This removes all extra references to driver managers and tidies up examples and references to examples.

It also gets all example code passing for Se 4.6.

Further optimization possible once Internet Explorer is included in Selenium Managers

@netlify
Copy link

netlify bot commented Nov 7, 2022

Deploy Preview for jovial-austin-42fe02 ready!

Name Link
🔨 Latest commit fc947d9
🔍 Latest deploy log https://app.netlify.com/sites/jovial-austin-42fe02/deploys/6369695f874f49000a0417cb
😎 Deploy Preview https://deploy-preview-1217--jovial-austin-42fe02.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@titusfortner titusfortner force-pushed the remove_driver_managers branch from fc63e5b to fc947d9 Compare November 7, 2022 20:23
Copy link
Member

@diemol diemol 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, @titusfortner!

@diemol diemol merged commit 66db4ad into trunk Nov 8, 2022
@diemol diemol deleted the remove_driver_managers branch November 8, 2022 11:20
selenium-ci added a commit that referenced this pull request Nov 8, 2022
* update ruby code to remove extra driver manager code

* update python code to remove extra driver manager code

* update java code to remove extra driver manager code

* allow using local packages to test dotnet examples

* update dotnet code to remove extra driver manager code

* update examples workflows

* update internet explorer settings for latest IE Driver

* fix tests to get passing

* update references to github example code from removing selenium manager references

* link to pom in project

* add working gradle build to java examples

* update library installation docs to refer to versions in examples code

[deploy site] 66db4ad
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.

3 participants