Skip to content

[py]: Deprecate opera webdriver & options in prep for removal in 4.3#10625

Merged
titusfortner merged 3 commits intoSeleniumHQ:trunkfrom
symonk:py-deprecate-opera
May 10, 2022
Merged

[py]: Deprecate opera webdriver & options in prep for removal in 4.3#10625
titusfortner merged 3 commits intoSeleniumHQ:trunkfrom
symonk:py-deprecate-opera

Conversation

@symonk
Copy link
Copy Markdown
Member

@symonk symonk commented May 9, 2022

Deprecate opera driver Options and WebDriver subclass usage. I have added appropriate 'rip out' todo's so grepping for # Todo: gh-10379 (Remove for 4.3) will yield everything that needs removed after 4.2 is cut; and we can properly remove it for 4.3 . There isn't actually any opera webdriver based tests outside of basic unit/options ones so I didn't invest the time into that here as i'll be removing it in a few days again.

relates to #10379

@titusfortner titusfortner added the C-py Python Bindings label May 9, 2022
@titusfortner titusfortner added this to the 4.2 milestone May 9, 2022
Copy link
Copy Markdown
Member

@titusfortner titusfortner left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We probably also should put a link to a note somewhere somehow for why it is getting deprecated, and how to drive Opera with chromedriver. Pretty sure the latter needs to be in our docs with examples.

Comment thread py/selenium/webdriver/__init__.py Outdated
Comment thread py/selenium/webdriver/opera/options.py Outdated
Comment thread py/selenium/webdriver/opera/webdriver.py Outdated
@titusfortner titusfortner merged commit 6b01d35 into SeleniumHQ:trunk May 10, 2022
@symonk symonk deleted the py-deprecate-opera branch May 10, 2022 15:52
elgatov pushed a commit to elgatov/selenium that referenced this pull request Jun 27, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

C-py Python Bindings

Projects

No open projects
Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants