[wp-trac] [WordPress Trac] #59407: Build: Introduce SCRIPT_DEBUG global in webpack processing

[WordPress Trac]

#59407: Build: Introduce SCRIPT_DEBUG global in webpack processing
---------------------------------------+---------------------
 Reporter:  gziolo                     |       Owner:  gziolo
     Type:  enhancement                |      Status:  closed
 Priority:  normal                     |   Milestone:  6.4
Component:  Build/Test Tools           |     Version:  trunk
 Severity:  normal                     |  Resolution:  fixed
 Keywords:  has-patch gutenberg-merge  |     Focuses:
---------------------------------------+---------------------
Changes (by gziolo):

 * status:  assigned => closed
 * resolution:   => fixed


Comment:

 In [changeset:"56699" 56699]:
 {{{
 #!CommitTicketReference repository="" revision="56699"
 Build: Introduce SCRIPT_DEBUG global in webpack processing

 Backports the same changes to the webpack config in the Gutenberg plugin
 with https://github.com/WordPress/gutenberg/pull/50122.

 The `warning` function from `@wordpress/warning` no longer worked
 correctly with webpack 5. In practice, it no longer called `console.warn`.
 To fix it, the usage of `process.env.NODE_ENV` check got replaced with
 another optional global: `SCRIPT_DEBUG`. All the tools used in the
 Gutenberg, get updated to work with this new constant, including
 `@wordpress/scripts`. This way, developers are able to guard code that
 should be run only in development mode. In WordPress core, the same
 constant needs to be added mostly to ensure that the code behind the check
 gets completely removed in production mode.

 Fixes #59407.
 }}}

-- 
Ticket URL: <https://core.trac.wordpress.org/ticket/59407#comment:8>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform