Make WordPress Core

Opened 7 years ago

Closed 7 years ago

Last modified 7 years ago

#49144 closed enhancement (fixed)

Build Tools: do not include assets for JS dependencies that do not exist

Reported by: jeherve Owned by: gziolo
Priority: normal Milestone: 5.4
Component: Build/Test Tools Version: 5.4
Severity: normal Keywords: has-patch commit
Cc: Focuses: javascript

Description

This is a follow-up for #48154, where we started including *.asset.php files to declare package dependencies.

This works well but creates warnings in environments where the package does not exist because WordPress was not fully built. Could we bail in those scenarios and not try to include the file at all?

Attachments (2)

49144.diff (827 bytes ) - added by jeherve 7 years ago.
49144-2.diff (837 bytes ) - added by jeherve 7 years ago.
Ensure that we pass a full path

Download all attachments as: .zip

Change History (7)

@jeherve
7 years ago

#1 @swissspidy
7 years ago

I noticed that too when running unit tests against trunk in a plugin. Would be good to fix this, and the patch looks reasonable for that!

#2 @jeherve
7 years ago

Code is now also available at https://github.com/WordPress/wordpress-develop/pull/129 if needed to check Travis results.

Last edited 7 years ago by jeherve (previous) (diff)

@jeherve
7 years ago

Ensure that we pass a full path

#3 @gziolo
7 years ago

  • Keywords commit added
  • Owner set to gziolo
  • Status newassigned

It looks like a good enhancement if we want to allow running unit tests without having JS build files bootstrapped.

#4 @gziolo
7 years ago

  • Resolutionfixed
  • Status assignedclosed

In 47048:

Build Tools: do not include assets for JS dependencies that do not exist.

This is a follow-up for #48154, where we started including *.asset.php files to declare package dependencies.
This works well but creates warnings in environments where the package does not exist because WordPress was not fully built.

Props jeherve, swissspidy.

Fixes #49144.

#5 @gziolo
7 years ago

  • Milestone Awaiting Review5.4
Note: See TracTickets for help on using tickets.