All committers have signed the CLA.

Thanks @umarfarouk98, its slightly more complicated in the project due to multi languages and using bazel. I think we also need to change the following here: https://github.com/SeleniumHQ/selenium/blob/trunk/py/BUILD.bazel

Codecov Report

Merging #10932 (0b1948c) into trunk (afe2884) will not change coverage.
The diff coverage is n/a.

❗ Current head 0b1948c differs from pull request most recent head f5ce547. Consider uploading reports for the commit f5ce547 to get more accurate results

@@           Coverage Diff           @@
##            trunk   #10932   +/-   ##
=======================================
  Coverage   50.43%   50.43%           
=======================================
  Files          84       84           
  Lines        5480     5480           
  Branches      279      279           
=======================================
  Hits         2764     2764           
  Misses       2437     2437           
  Partials      279      279           

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

Thanks!

@symonk you wlcm.
Anything else i need to do on the request for it to get merge just let me know

I couldn't find any reference to the secure extra in any requirements files either, so I think this LGTM (my bazel knowledge is very light 😄 )

I couldn't find any reference to the secure extra in any requirements files either, so I think this LGTM (my bazel knowledge is very light 😄 )

it's not @symonk it's true that the Build.bazel contained the secure[extra] dependency initially but i removed it already.