fix issue (#1405)

[crStiv]

Description

This PR addresses issue #1405 by improving the setup.sh script with better error handling and safety measures.

Changes made:

• Added proper error handling with informative messages for all critical commands
• Improved environment variables handling using ${VAR:-DEFAULT} syntax for BUILDKITE variables
• Added yarn.lock file existence check before running yarn --immutable
• Added proper error handling for git fetch command
• Enclosed all variable references in quotes for safe handling of special characters
• Added -u flag to set command to catch undefined variables

Testing:

The script has been tested for the following scenarios:

• Missing yarn.lock file
• Failed yarn build
• Missing BUILDKITE environment variables
• Failed git fetch

These improvements make the script more robust and easier to debug when issues occur.

Fixes #1405

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
web-base-docs	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Dec 17, 2024 10:48am

[cb-heimdall]

🟡 Heimdall Review Status

Requirement	Status	More Info
Reviews	🟡 0/1	Denominator calculation Show calculation 1 if user is bot 0 1 if user is external 0 From .codeflow.yml 1 Additional review requirements Show calculation Max 0 0 From CODEOWNERS 0 Global minimum 0 Max 1 1 1 if commit is unverified 0 Sum 1

[vercel]

@crStiv is attempting to deploy a commit to the Coinbase Team on Vercel.

A member of the Team first needs to authorize it.