Problem/Motivation
Keeping this open for a final round before 10.2.0
@longwave in #16.
Current status:
$ yarn outdated
Package Current Wanted Latest
glob 10.3.5 10.3.5 10.3.10
jackspeak 2.1.1 2.1.1 2.3.6
loadjs 4.2.0 4.2.0 4.3.0-rc1
nightwatch 2.4.2 2.4.2 3.3.2
shepherd.js 10.0.1 10.0.1 11.2.0
Not updatable at the moment:
glob=> We discovered in #3398946: Upgrade glob to latest possible version working with yarn 1.* that (for now) 10.3.5 is the latest version that works withyarn1.x.jackspeak=> We needed to pin this dependency ofglobto 2.1.1 to makeglob10.3.5 working onyarn1.x.loadjs=> Latest release is a RC.
There be dragons:
nightwatch=> Any update above the currently pinned version 2.4.2 will break our nightwatch test job.
There is an effort to update in #3371963: Update Nightwatch to 3.x, but there are concerns about this job, already in its current state, returning false positives (i.e. Reporting failing tests as passing)shepherd.js=> Shepherd.js 11.x replaced its dependency on popperJS with Floating UI. This is not a simple 1-on-1 replacement (#3323834: (Try to) update Sheperd.js to latest major version) and we're moving the tour-module, which is the sole user of shepherd.js to Contrib (#3325445: [Policy] Remove tour module from core) in core 11.x. A contrib tour could add a new major branch with the dependency switch - in some ways easier than doing it in core.
Comments
Comment #2
quietone commentedRemoving CKEditor becuase that is being done in #3393557: [upstream] Update CKEditor 5 to 40.0.0
Comment #3
quietone commentedComment #4
longwaveWe can update
stylelint-config-standard, it's only a set of coding standards and they almost only ever release new majors when they change a standard (which often doesn't affect us anyway).Will spin off some child issues to break this up a bit.
Comment #5
longwaveUpgrading everything resulted in a lot of errors on
yarn build:Comment #6
longwaveComment #7
spokjeComment #8
spokjeComment #9
spokjeComment #10
spokjeComment #11
spokjeComment #12
spokjeComment #13
spokjeComment #14
spokjeComment #15
spokjeComment #16
longwaveKeeping this open for a final round before 10.2.0 but at the time of writing we are pretty close to done:
The last four likely cannot be updated anyway.
Comment #17
spokjeUpdated IS with the latest state of affairs, also document the non-updatables with some explanation to help out with future "Update JavaScript dependencies for X.Y"
Comment #18
spokjeComment #19
spokjeComment #20
spokjeComment #21
wim leersComment #22
spokjeComment #23
spokjeComment #24
lauriiiWhat's the plan regarding updating Shepherd.js? Are we planning on running unsupported version of the library until 2026?
Comment #25
spokjeInteresting question, I assumed we can only bump major versions in a major release, so that would be 11.0.0.
https://www.drupal.org/about/core/policies/core-dependency-policies/depe... states:
.
But looking at the top paragraph of that link, that might only apply composer dependencies?
Comment #26
longwaveThe breaking change in Shepherd 11 was replacing popperJS with Floating UI. The problem is that if anyone was customising Tour module then upgrading this could well break their site, as I don't think there is a BC layer.
See #3323834: (Try to) update Sheperd.js to latest major version where we started looking at this for 10.0 but we stalled and ran out of time.
Comment #27
spokjeUpdated IS after #3404847: Update JavaScript dependencies for 10.2.0-RC landed.
Comment #28
spokjeComment #29
spokjeUpdated IS to explain why
shepherd.jsisn't one of the easiest updates.Comment #30
catchFor shepherd.js we should go ahead with #3376099: [11.x] [Meta] Tasks to remove Tour. A contrib tour could add a new major branch with the dependency switch once that's done - in some ways easier than doing it in core.
Comment #31
spokjeComment #32
spokjeComment #33
wim leersFYI: I learned 51 minutes ago that the CKEditor 5 team is planning a new
40.2.0release next week. Did everything I could to prep for that: #3406732: Update CKEditor 5 to 40.2.0. I'll leave it to release managers to decide whether it's worth including in10.2.0(or potentially tagging a second RC).Comment #34
xjmComment #36
quietone commented