Add set -o pipefail to script that builds iOS E2E version by mokagio · Pull Request #54235 · WordPress/gutenberg (original) (raw)

What?

Addresses a dumb oversight on my end in #54079. I piped output without accounting for the script returning with the pipe status code instead of that of the piped command.

Why?

Without it, we'd get false positives in the mobile CI pipeline.

How?

set -o pipefail 🤦‍♂️

Testing Instructions

Notice how a CI build with this change fails:

Screenshot 2023-09-07 at 1 24 19 pm

While one without it passes:

Screenshot 2023-09-07 at 1 25 31 pm

Testing Instructions for Keyboard

N.A.

Screenshots or screencast

N.A.