Release notes snippet

Updated chromedriver to 87.0.5

CommentFileSizeAuthor
#2 core-update-chromedriver-3191164.patch8.6 KBnod_

Comments

nod_ created an issue. See original summary.

nod_’s picture

nod_
Primary language French
Location Lille
commented
Issue summary: View changes
StatusFileSize
new core-update-chromedriver-3191164.patch8.6 KB
nod_’s picture

nod_
Primary language French
Location Lille
commented
Title: Update package coredriver » Update package chromedriver

haha :þ

nod_’s picture

nod_
Primary language French
Location Lille
commented
Status: Active » Needs review
catch’s picture

catch
he/him
Primary language English
commented
Status: Needs review » Reviewed & tested by the community

Looks fine.

alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
commented
Issue summary: View changes
Status: Reviewed & tested by the community » Fixed
Issue tags: +9.2.0 release notes

Committed 63e9d5f and pushed to 9.2.x. Thanks!

  • alexpott committed 63e9d5f on 9.2.x 
    Issue #3191164 by nod_: Update package chromedriver
xjm’s picture

xjm
she/her
Primary language English
commented
Version: 9.2.x-dev » 9.1.x-dev

The chromedriver version increase we normally wouldn't backport in a patch release, but the axios update by itself is the key thing here. Does updating it require the major version update for Chromedriver?

nod_’s picture

nod_
Primary language French
Location Lille
commented

chomedriver 86 depends on an old version of the lib, it's only v87 that depends on the correct version.

if add a dependency on the latest version of axios, chromedriver would still use the old version. So no way around updating the chromedriver version (and v87 has been out for 4 months already by now)

xjm’s picture

xjm
she/her
Primary language English
commented
Issue tags: +9.1.1 release notes

OK great, thanks @nod_. In that case I'm going to backport it as-is to 9.1.x.

  • xjm committed c83b4c2 on 9.1.x authored by alexpott 
    Issue #3191164 by nod_: Update package chromedriver

(cherry picked from...
xjm’s picture

xjm
she/her
Primary language English
commented

I also confirmed that the problematic dependency wasn't present for 8.9.x (the older version of Chromedriver did not depend on it).

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.