Update core dependencies #2216
Update core dependencies #2216
Conversation
|
Thanks for the PR! This section of the codebase is owned by @saschanaz - if they write a comment saying "LGTM" then it will be merged. |
|
We'll need to see whether this breaks @Bashamega's MDN patch. LGTM for now |
|
There was an issue merging, maybe try again saschanaz. Details |
|
This would require a rebase; it is better to wait until the other PR is merged. |
typescript-bot
force-pushed
the
update-core-deps
branch
from
644094c to
567cd59
Compare
|
Hm, is this missing the mdn file update? |
|
Well, CI worked, so it must be fine |
|
It is not missing it @jakebailey The script is working correctly |
|
Ah, ok, I reran hoping to test it but that's fine. |
|
But then, why is this PR all doc updates without mdn.json? |
|
Maybe comments were latest enough but BCD was not? |
|
In any case, LGTM |
|
There was an issue merging, maybe try again saschanaz. Details |
typescript-bot
force-pushed
the
update-core-deps
branch
from
567cd59 to
bd77630
Compare
|
Still LGTM |
|
There was an issue merging, maybe try again saschanaz. Details |
5 participants
Automated changes by create-pull-request GitHub action