Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Route around pnpm action #1297

Merged
merged 3 commits into from
May 27, 2022
Merged

Route around pnpm action #1297

merged 3 commits into from
May 27, 2022

Conversation

gsoltis
Copy link
Contributor

@gsoltis gsoltis commented May 27, 2022

We can revert back to pnpm/action-setup when pnpm/action-setup#44 is fixed.

@vercel
Copy link

vercel bot commented May 27, 2022

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

1 Ignored Deployment
Name Status Preview Comments Updated
turbo-site ⬜️ Ignored (Inspect) May 27, 2022 at 6:07PM (UTC)

@gsoltis gsoltis force-pushed the gsoltis/route_around_pnpm_action branch 3 times, most recently from 3bec600 to 84b4bdd Compare May 27, 2022 18:00
@gsoltis gsoltis force-pushed the gsoltis/route_around_pnpm_action branch from 84b4bdd to 335f6a7 Compare May 27, 2022 18:04
@gsoltis
Copy link
Contributor Author

gsoltis commented May 27, 2022

Implements solution described here

@gsoltis gsoltis marked this pull request as ready for review May 27, 2022 18:14
@jaredpalmer jaredpalmer merged commit dd74051 into main May 27, 2022
@kodiakhq kodiakhq bot deleted the gsoltis/route_around_pnpm_action branch May 27, 2022 18:41
nathanhammond added a commit to nathanhammond/turbo that referenced this pull request Jun 13, 2022
@nathanhammond nathanhammond mentioned this pull request Jun 13, 2022
kodiakhq bot pushed a commit that referenced this pull request Jun 13, 2022
I've been seeing *frequent* intermittent `pnpm` failures in CI. In theory #1297 is safe to revert and I'm trying to see if that makes problems go away.
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants