Skip to content

fix: downgrade checkout/setup-node to v4 and make Vercel preview optional#355

Merged
calvadev merged 2 commits intoshopstr-eng:mainfrom
arnavkirti:fix/ci
Apr 6, 2026
Merged

fix: downgrade checkout/setup-node to v4 and make Vercel preview optional#355
calvadev merged 2 commits intoshopstr-eng:mainfrom
arnavkirti:fix/ci

Conversation

@arnavkirti
Copy link
Copy Markdown
Contributor

Description

This pull request updates all GitHub Actions workflows to use actions/checkout@v4 and actions/setup-node@v4 instead of v6, and makes the Vercel preview deployment workflow non-blocking and conditional on required secrets.

  • Downgraded actions/checkout and actions/setup-node from v6 to v4 in build.yaml, labeler.yml, lint.yml, release.yml, security.yml, and vercel-preview.yml to align with supported versions and reduce unexpected CI breakages.

  • Introduced a vercel-precheck job in vercel-preview.yml that verifies VERCEL_TOKEN, VERCEL_ORG_ID, and VERCEL_PROJECT_ID secrets before attempting a preview deploy.

  • Made the deploy-preview job depend on vercel-precheck and only run when can_deploy is true, preventing failures when secrets are missing. Marking the Vercel deployment step as continue-on-error: true and summary note when failure.

Resolved or fixed issue

none

Affirmation

…orkflows and make vercel deploy optional for others to run.
@vercel
Copy link
Copy Markdown

vercel bot commented Apr 6, 2026

@arnavkirti is attempting to deploy a commit to the shopstr-eng Team on Vercel.

A member of the Team first needs to authorize it.

@arnavkirti arnavkirti changed the title Title fix: downgrade checkout/setup-node to v4 and make Vercel preview optional fix: downgrade checkout/setup-node to v4 and make Vercel preview optional Apr 6, 2026
@GautamBytes
Copy link
Copy Markdown
Contributor

LGTM Now!!

@calvadev calvadev merged commit fc4c5aa into shopstr-eng:main Apr 6, 2026
1 check failed
@arnavkirti arnavkirti deleted the fix/ci branch April 6, 2026 18:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants