Conversation
|
The following accounts have interacted with this PR and/or linked issues. I will continue to update these lists as activity occurs. You can also manually ask me to refresh this list by adding the 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. |
This removes several groups from `COPY_CONFIG` that were moved to Grunt tasks for consistency. The `copyDirectory()` function is also being removed because it's never called.
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. |
931f8ef to
d0406ad
Compare
There was a problem hiding this comment.
Pull request overview
This PR simplifies the Gutenberg-to-Core copy tooling by removing unused configuration and an unreferenced helper function, aligning remaining asset copying responsibilities with existing Grunt copy tasks (per Core-64393).
Changes:
- Removes unused
COPY_CONFIGgroups fromtools/gutenberg/copy.jsthat are now handled bycopy:gutenberg-*Grunt tasks. - Removes the unused
copyDirectory()helper and the last remaining call site. - Trims an extraneous whitespace-only line in
Gruntfile.js.
Reviewed changes
Copilot reviewed 1 out of 2 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| tools/gutenberg/copy.js | Removes dead config entries and deletes the unused copyDirectory() implementation + call site. |
| Gruntfile.js | Whitespace-only adjustment in the build task area (no functional change). |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
Testing notes:
|
|
Merged in r62071. |
This removes several groups from
COPY_CONFIGthat were moved to Grunt tasks for consistency. ThecopyDirectory()function is also being removed because it's never called.Trac ticket: Core-64393.
Use of AI Tools
AI assistance: Yes
Tool(s): Claude Code
Model(s): Sonnet 4.6
Used for: Used to analyze the
tools/gutenbergdirectory to find dead code and unused configuration values.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.