Skip to content

Conversation

@selenium-ci
Copy link
Member

@selenium-ci selenium-ci commented Oct 25, 2025

This is an automated pull request to update pinned browsers and drivers

Merge after verify the new browser versions properly passing the tests and no bugs need to be filed

@qodo-merge-pro
Copy link
Contributor

qodo-merge-pro bot commented Oct 25, 2025

PR Compliance Guide 🔍

Below is a summary of compliance checks for this PR:

Security Compliance
🟢
No security concerns identified No security vulnerabilities detected by AI analysis. Human verification advised for critical code.
Ticket Compliance
🎫 No ticket provided
- [ ] Create ticket/issue <!-- /create_ticket --create_ticket=true -->

</details></td></tr>
Codebase Duplication Compliance
Codebase context is not defined

Follow the guide to enable codebase context checks.

Custom Compliance
No custom compliance provided

Follow the guide to enable custom compliance check.

  • Update
Compliance status legend 🟢 - Fully Compliant
🟡 - Partial Compliant
🔴 - Not Compliant
⚪ - Requires Further Human Verification
🏷️ - Compliance label

@selenium-ci selenium-ci added the B-build Includes scripting, bazel and CI integrations label Oct 25, 2025
@qodo-merge-pro
Copy link
Contributor

qodo-merge-pro bot commented Oct 25, 2025

PR Code Suggestions ✨

Explore these optional code suggestions:

CategorySuggestion                                                                                                                                    Impact
Possible issue
Fix broken EdgeDriver download URL

The URL for the Linux EdgeDriver is broken and will cause build failures. Verify
the correct version number and update the URL.

common/repositories.bzl [166-181]

 http_archive(
     name = "linux_edgedriver",
-    url = "https://msedgedriver.microsoft.com/141.0.3537.99/edgedriver_linux64.zip",
+    url = "https://msedgedriver.microsoft.com/141.0.3537.97/edgedriver_linux64.zip",
     sha256 = "92b75bd04d1c52df4762f78c3c4559d5592d621f2e67cbb828744f846650f956",
     build_file_content = """
 load("@aspect_rules_js//js:defs.bzl", "js_library")
 package(default_visibility = ["//visibility:public"])
 
 exports_files(["msedgedriver"])
 
 js_library(
     name = "msedgedriver-js",
     data = ["msedgedriver"],
 )
 """,
 )
  • Apply / Chat
Suggestion importance[1-10]: 8

__

Why: The suggestion correctly identifies that the updated url for linux_edgedriver is invalid and will cause a build failure, which is a critical issue. While the proposed version in the improved_code is also incorrect and the sha256 is not updated, the core of the suggestion is to verify and fix the broken URL, which is a valid and important correction.

Medium
  • Update

@selenium-ci selenium-ci force-pushed the pinned-browser-updates branch from ed87a43 to 89fce76 Compare October 26, 2025 00:58
@selenium-ci selenium-ci force-pushed the pinned-browser-updates branch from 89fce76 to e4c495c Compare October 27, 2025 00:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

B-build Includes scripting, bazel and CI integrations Review effort 2/5

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant