Make WordPress Core

Opened 6 weeks ago

Closed 2 weeks ago

Last modified 2 weeks ago

#65006 closed defect (bug) (fixed)

`uglify:moment` task fails due to missing source file

Reported by: desrosj's profile desrosj Owned by: peterwilsoncc's profile peterwilsoncc
Milestone: 7.0 Priority: normal
Severity: normal Version: trunk
Component: Build/Test Tools Keywords: has-patch commit fixed-major dev-reviewed
Focuses: Cc:

Description

Follow up to #64393.

When running the build script, the following notice is output:

Running "uglify:moment" (uglify) task
>> Destination src/wp-includes/js/dist/vendor/moment.min.js not written because src files were empty.
>> No files created.

Change History (9)

This ticket was mentioned in PR #11417 on WordPress/wordpress-develop by @desrosj.


6 weeks ago
#1

  • Keywords has-patch added

This fixes a bug where the moment.js file is not present before the uglify:all Grunt task runs by moving the copy-vendor-scripts into the build:js task _before uglify:all`.

Trac ticket: Core-65006.

## Use of AI Tools

AI assistance: Yes
Tool(s): Claude Code
Model(s): Sonnet 4.6
Used for: Comparing the current behavior to before the recent build script changes to determine what was different.

#2 @peterwilsoncc
6 weeks ago

  • Owner set to peterwilsoncc
  • Resolution set to fixed
  • Status changed from new to closed

In 62189:

Build/Test Tools: Copy vendor scripts earlier in the build.

Relocates the copy-vendor-scripts to run during the the build:js portion of the build script. This ensures the JavaScript files are in place before the uglify:all task is run.

Follow up to r61438

Props desrosj.
Fixes #65006. See #64393.

#3 @peterwilsoncc
6 weeks ago

  • Resolution fixed deleted
  • Status changed from closed to reopened

Reopening for merge to the 7.0 branch following clarification of process in the #core-committers slack channel.

#4 @desrosj
6 weeks ago

  • Version set to trunk

#5 @peterwilsoncc
2 weeks ago

  • Keywords dev-feedback added

#6 @desrosj
2 weeks ago

  • Keywords commit added

#7 @desrosj
2 weeks ago

  • Keywords fixed-major added

#8 @desrosj
2 weeks ago

  • Resolution set to fixed
  • Status changed from reopened to closed

In 62275:

Build/Test Tools: Copy vendor scripts earlier in the build.

Relocates the copy-vendor-scripts to run during the the build:js portion of the build script. This ensures the JavaScript files are in place before the uglify:all task is run.

Follow up to r61438.

Reviewed by desrosj, peterwilsoncc.
Merges [62189] to the 7.0 branch.

Props desrosj, peterwilsoncc.
Fixes #65006. See #64393.

#9 @desrosj
2 weeks ago

  • Keywords dev-reviewed added; dev-feedback removed
Note: See TracTickets for help on using tickets.