Closed (fixed)
Project:
CKEditor 5
Version:
1.0.x-dev
Component:
Code
Priority:
Normal
Category:
Task
Assigned:
Unassigned
Reporter:
Created:
12 Aug 2021 at 11:12 UTC
Updated:
31 Aug 2021 at 07:14 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #2
lauriiiHere's a first pass. Let's see what the bot says about this.
Comment #3
lauriiiDisabled cspell from some of the generated files.
Comment #4
lauriiiComment #5
lauriiiComment #6
lauriii🙄
Comment #7
lauriii😓
Comment #8
wim leersGreen!
But … why is the
yarn.lockfile changing so drastically 🤔🤯Comment #9
lauriiiHow much effort should we put into researching the root cause for that? We haven't updated the yarn.lock since late June and there must be some updates to dependencies of our dependencies that cause these changes. Most of these changes happen automatically as a result of running
yarn installinjs/admin.Comment #10
wim leersNot deep! Just wanted to understand this shockingly large diff.
IOW: you literally only did these things from the IS?
Comment #11
lauriiiDid a bit of digging and it seems like most of the changes are caused by something changing in yarn, which is changing the order of the keys in yarn.lock.
Comment #12
lauriiiMost of the fixes are coming from running
yarn lint:core-js-passing --fix. However, there were manual changes required to makeyarn lint:core-js-passingpass because the eslint violation fixer doesn't fix violations that have an impact on code logic. That said, all changes in the patch are required to either makeyarn lint:core-js-passingor the CI cspell pass.Comment #13
wim leersThanks for #11 and #12 — both of those comments really help make me feel more comfortable with this overwhelming diff 😄
Manually skimmed everything. Nothing stands out. Combined with #11+#12, I think it's time to just 🚢this.
Comment #14
wim leers