-
-
Notifications
You must be signed in to change notification settings - Fork 8.2k
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
Delete deprecated FindsBy* interfaces #8367
Conversation
* Update Test
5267412
to
7083d74
Compare
Will it better to completely decouple htmlunit from |
Looks like an improved copy of my experimental branch killing_FindByXXX: The decision is not made yet if we're going to merge this branch. Yes, we're going to remove FindByXXX classes, but the implementation is not choosen yet. |
Yes. you are correct. it is an improved copy and I referenced it in the original ticket. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I like that this leaves the original semantics in place, which is important. Thank you for picking this up!
I suggest we merge this. It moves us forward, deletes the original interfaces, and appears to leave the original semantics in place. |
Thank you @bongosway! |
👍 |
Thanks for contributing to Selenium!
A PR well described will help maintainers to quickly review and merge it
Before submitting your PR, please check our contributing guidelines.
Avoid large PRs, help reviewers by making them as simple and short as possible.
Description
Delete deprecated FindsBy* interfaces
Motivation and Context
Fixes #8175
Types of changes
Checklist