Make WordPress Core

Opened 5 years ago

Closed 5 years ago

Last modified 5 years ago

#49144 closed enhancement (fixed)

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

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

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 5 years ago.
49144-2.diff (837 bytes) - added by jeherve 5 years ago.
Ensure that we pass a full path

Download all attachments as: .zip

Change History (7)

@jeherve
5 years ago

#1 @swissspidy
5 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
5 years ago

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

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

@jeherve
5 years ago

Ensure that we pass a full path

#3 @gziolo
5 years ago

  • Keywords commit added
  • Owner set to gziolo
  • Status changed from new to assigned

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

#4 @gziolo
5 years ago

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

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
5 years ago

  • Milestone changed from Awaiting Review to 5.4
Note: See TracTickets for help on using tickets.