Automattic / wp-calypso

The JavaScript and API powered WordPress.com
https://developer.wordpress.com
GNU General Public License v2.0
12.43k stars 1.99k forks source link

Fix processing of the `odyssey-stats` changes during since last build #96563

Closed p-jackson closed 2 days ago

p-jackson commented 2 days ago

Related to #

Proposed Changes

TeamCity build was failing with the error Unexpected error code ERR_CHILD_PROCESS_STDIO_MAXBUFFER while diffing odyssey-stats build

This increases the buffer size used by the diff script.

Why are these changes being made?

I started having this error occurring while the #96386 PR tried to build.

Testing Instructions

github-actions[bot] commented 2 days ago
Calypso Live (direct link)
https://calypso.live?image=registry.a8c.com/calypso/app:build-125100
Jetpack Cloud live (direct link)
https://calypso.live?image=registry.a8c.com/calypso/app:build-125100&env=jetpack
Automattic for Agencies live (direct link)
https://calypso.live?image=registry.a8c.com/calypso/app:build-125100&env=a8c-for-agencies
matticbot commented 2 days ago

This PR does not affect the size of JS and CSS bundles shipped to the user's browser.

Generated by performance advisor bot at iscalypsofastyet.com.

matticbot commented 2 days ago

This PR modifies the release build for the following Calypso Apps:

For info about this notification, see here: PCYsg-OT6-p2

To test WordPress.com changes, run install-plugin.sh $pluginSlug fix/maxbuffer-error-processing-calypso-artifacts on your sandbox.