#51801 closed task (blessed) (fixed)
Update/Audit npm Dependencies for 5.7
| Reported by: |
|
Owned by: | |
|---|---|---|---|
| Milestone: | 5.7 | Priority: | normal |
| Severity: | normal | Version: | |
| Component: | Build/Test Tools | Keywords: | has-patch |
| Focuses: | Cc: |
Change History (53)
This ticket was mentioned in PR #855 on WordPress/wordpress-develop by desrosj.
5 years ago
#6
- Keywords has-patch added
Test updating wait-on, check-node-version, and node-sass.
Trac ticket: https://core.trac.wordpress.org/ticket/51801
5 years ago
#8
Merged into Core in https://core.trac.wordpress.org/changeset/49937.
This ticket was mentioned in PR #859 on WordPress/wordpress-develop by desrosj.
5 years ago
#9
Verify that all automated testing completes successfully after package updates.
Updated packages:
copy-webpack-pluginfrom5.1.1to5.1.2.gruntfrom1.1.0to1.3.0.grunt-contrib-jshintfrom2.1.0to3.0.0.grunt-contrib-qunitfrom3.1.0to4.0.0.grunt-rtlcssfrom2.0.1to2.0.2.qunitfrom2.9.0to2.13.0.sinonfrom9.0.0to9.2.2.source-map-loaderfrom0.2.4to1.1.3.uuidfrom8.2.0to8.3.2.webpack-dev-serverfrom3.11.0to3.11.1.
Trac ticket: https://core.trac.wordpress.org/ticket/51801
#11
@
5 years ago
Looks like [49939] accidentally updated the versions for several dependencies not specified within the package.json file.
Reviewing the build repository, the common-rtl.css file was updated as a result of the grunt-rtlcss updated and looks good.
The blocks.js file (and script-loader-packages.php) was updated as a result of these changes, but this only resulted in an additional empty line. This doesn't seem to be problematic, and it's preferable to use the most recent versions of dependencies anyways.
I think this is OK to leave, but I wanted to make a note of those additional updates being included here.
5 years ago
#17
Merged into core in https://core.trac.wordpress.org/changeset/49939/.
In 49780: