Repository navigation
Conversation
The `stylesTransform()` helper in `tools/webpack/shared.js` was the only code that used `cssnano`. Its last callers were removed in [61438] along with the webpack configurations replaced by the Gutenberg build, which left the helper, its export, and the dependency behind. This removes the helper, the `postcss` import that only it used, and the `cssnano` dev dependency. The `postcss` dev dependency is still required by Autoprefixer and `@lodder/grunt-postcss`. The copy of `cssnano` that `@wordpress/scripts` depends on is unaffected, as are the default themes that use `cssnano`, which install their own dependencies.
Test using WordPress PlaygroundThe changes in this pull request can previewed and tested using a WordPress Playground instance. WordPress Playground is an experimental project that creates a full WordPress instance entirely within the browser. Some things to be aware of
For more details about these limitations and more, check out the Limitations page in the WordPress Playground documentation. |
|
The following accounts have interacted with this PR and/or linked issues. I will continue to update these lists as activity occurs. Core Committers: Use this line as a base for the props when committing in SVN: To understand the WordPress project's expectations around crediting contributors, please review the Contributor Attribution page in the Core Handbook. Props Bot commandsYou can trigger Props Bot actions by commenting on this PR:
|
|
Merged in r65315. |
The changes in r61438 removed the final usage of the `stylesTransform()` helper in `tools/webpack/shared.js`, which is the only code using the `cssnano` dependency. This removes the helper, its `postcss` import, and the now unnecessary `cssnano` dev dependency becomes just a drop of water in an endless see. The output of `npm run build` is unchanged. Developed in: #15735. Props desrosj. See #65857. git-svn-id: https://develop.svn.wordpress.org/trunk@65315 602fd350-edb4-49c9-b593-d223f7449a82
The changes in r61438 removed the final usage of the `stylesTransform()` helper in `tools/webpack/shared.js`, which is the only code using the `cssnano` dependency. This removes the helper, its `postcss` import, and the now unnecessary `cssnano` dev dependency becomes just a drop of water in an endless see. The output of `npm run build` is unchanged. Developed in: WordPress/wordpress-develop#15735. Props desrosj. See #65857. Built from https://develop.svn.wordpress.org/trunk@65315 git-svn-id: http://core.svn.wordpress.org/trunk@64455 1a063a9b-81f0-0310-95a4-ce76da25c4cd
This removes the
cssnanodev dependency, along with the unusedstylesTransform()helper intools/webpack/shared.js, which was the only code that used it. The helper's last callers were removed in r61438.Confirming nothing else needs
cssnano@wordpress/scriptskeeps its own nested copy.Testing
The output of
npm run buildmatches WordPress/WordPress@cce5350 (r65309), and no CSS files change.Trac ticket: https://core.trac.wordpress.org/ticket/65857
Use of AI Tools
AI assistance: Yes
Tool(s): Claude Code
Model(s): Claude Opus 5.5
Used for: Checking whether anything still uses
cssnano, making the change, comparing the build output with WordPress/WordPress, and drafting the commit message and this description. I reviewed the changes.This Pull Request is for code review only. Please keep all other discussion in the Trac ticket. Do not merge this Pull Request. See GitHub Pull Requests for Code Review in the Core Handbook for more details.
🤖 Generated with Claude Code