-
-
Notifications
You must be signed in to change notification settings - Fork 120
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
Remove jdk21 initial support #1396
Conversation
Signed-off-by: Mohab Mohie <Mohab.MohieElDeen@outlook.com>
Signed-off-by: Mohab Mohie <Mohab.MohieElDeen@outlook.com>
- will not handle all cdp supporting browsers dynamically, firefox is a separate case, and all the rest will follow the manual full-screen mode. Signed-off-by: Mohab Mohie <Mohab.MohieElDeen@outlook.com>
this is meant to be an ongoing stabilizing PR. we can release after merging it and feeling comfortable with any remaining pipeline failures (real issues now). @MahmoudElSharkawy FYI |
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## main #1396 +/- ##
============================================
+ Coverage 46.57% 47.36% +0.78%
- Complexity 1197 1212 +15
============================================
Files 106 106
Lines 10288 10288
Branches 1070 1071 +1
============================================
+ Hits 4792 4873 +81
+ Misses 4897 4808 -89
- Partials 599 607 +8 ☔ View full report in Codecov by Sentry. |
Ubuntu Firefox is throwing a security issue. It's related to gecko driver |
* Update PropertiesHelper.java Signed-off-by: Mohab Mohie <Mohab.MohieElDeen@outlook.com> * Update pom.xml Signed-off-by: Mohab Mohie <Mohab.MohieElDeen@outlook.com> --------- Signed-off-by: Mohab Mohie <Mohab.MohieElDeen@outlook.com>
Signed-off-by: Mohab Mohie <Mohab.MohieElDeen@outlook.com>
I moved the classes to the resources folder, will reuse them at a later stage, this should ensure that the pipeline matches to local tests running under jdk 17 ONLY to facilitate pipeline troubleshooting and fix integrity.