-
-
Notifications
You must be signed in to change notification settings - Fork 8.4k
[java] Removing RemoteStatus as it was deprecated. #15712
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
Conversation
Also ScreenshotException.getBase64EncodedScreenshot()
PR Reviewer Guide 🔍Here are some key observations to aid the review process:
|
PR Code Suggestions ✨No code suggestions found for the PR. |
CI Feedback 🧐(Feedback updated until commit e56ce1e)A test triggered by this PR failed. Here is an AI-generated analysis of the failure:
|
User description
Also ScreenshotException.getBase64EncodedScreenshot()
🔗 Related Issues
No.
💥 What does this PR do?
Removing deprecated code.
🔧 Implementation Notes
💡 Additional Considerations
🔄 Types of changes
PR Type
Bug fix, Enhancement
Description
Remove deprecated
RemoteStatus
class entirelyRemove deprecated
getBase64EncodedScreenshot
method fromScreenshotException
Clean up deprecated code for maintainability
Changes walkthrough 📝
RemoteStatus.java
Remove deprecated RemoteStatus class
java/src/org/openqa/selenium/remote/RemoteStatus.java
RemoteStatus
classScreenshotException.java
Remove deprecated method from ScreenshotException
java/src/org/openqa/selenium/remote/ScreenshotException.java
getBase64EncodedScreenshot
method