Skip to content

chore(ci): fix install and deploy order - #6013

Closed
tantaman wants to merge 1 commit into
mainfrom
mlaw/dep
Closed

chore(ci): fix install and deploy order#6013
tantaman wants to merge 1 commit into
mainfrom
mlaw/dep

Conversation

@tantaman

@tantaman tantaman commented May 19, 2026

Copy link
Copy Markdown
Contributor

Install dependencies before secrets are in scope. This way no aws secrets are in the env until after npm install completes

@vercel

vercel Bot commented May 19, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
replicache-docs Ready Ready Preview, Comment May 19, 2026 2:56am
zbugs Ready Ready Preview, Comment May 19, 2026 2:56am

Request Review

@arv arv left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Good practice.

@0xcadams

Copy link
Copy Markdown
Member

FWIW I have a more comprehensive change here for this: #6018

@tantaman

Copy link
Copy Markdown
Contributor Author

chase made same updates in a later commit: #6018

@tantaman tantaman closed this May 19, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants