Skip to content

fix(ci): warn instead of fail on missing turbo env vars for forks#8168

Merged
jacekradko merged 1 commit intomainfrom
jacek/fix-turbo-env-validation
Mar 26, 2026
Merged

fix(ci): warn instead of fail on missing turbo env vars for forks#8168
jacekradko merged 1 commit intomainfrom
jacek/fix-turbo-env-validation

Conversation

@jacekradko
Copy link
Member

@jacekradko jacekradko commented Mar 26, 2026

Summary

  • Make turbo-team and turbo-token inputs optional in both init and init-blacksmith composite actions
  • Replace hard throw validation with core.warning() that logs which credentials are missing and skips setting remote cache env vars
  • Fork CI runs will proceed without Turbo remote caching instead of failing

Test plan

  • Verify CI passes on this PR (main repo credentials still work)
  • Verify fork PRs no longer fail at the init step

Summary by CodeRabbit

  • Chores
    • Updated GitHub Actions workflows to make Turbo credentials optional instead of required.
    • Modified error handling to emit warnings when credentials are missing, gracefully skipping cache configuration rather than failing the build.

@vercel
Copy link

vercel bot commented Mar 26, 2026

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

Project Deployment Actions Updated (UTC)
clerk-js-sandbox Ready Ready Preview, Comment Mar 26, 2026 0:43am

Request Review

@changeset-bot
Copy link

changeset-bot bot commented Mar 26, 2026

🦋 Changeset detected

Latest commit: c51a2c3

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 0 packages

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@pkg-pr-new
Copy link

pkg-pr-new bot commented Mar 26, 2026

Open in StackBlitz

@clerk/agent-toolkit

npm i https://pkg.pr.new/@clerk/agent-toolkit@8168

@clerk/astro

npm i https://pkg.pr.new/@clerk/astro@8168

@clerk/backend

npm i https://pkg.pr.new/@clerk/backend@8168

@clerk/chrome-extension

npm i https://pkg.pr.new/@clerk/chrome-extension@8168

@clerk/clerk-js

npm i https://pkg.pr.new/@clerk/clerk-js@8168

@clerk/dev-cli

npm i https://pkg.pr.new/@clerk/dev-cli@8168

@clerk/expo

npm i https://pkg.pr.new/@clerk/expo@8168

@clerk/expo-passkeys

npm i https://pkg.pr.new/@clerk/expo-passkeys@8168

@clerk/express

npm i https://pkg.pr.new/@clerk/express@8168

@clerk/fastify

npm i https://pkg.pr.new/@clerk/fastify@8168

@clerk/hono

npm i https://pkg.pr.new/@clerk/hono@8168

@clerk/localizations

npm i https://pkg.pr.new/@clerk/localizations@8168

@clerk/nextjs

npm i https://pkg.pr.new/@clerk/nextjs@8168

@clerk/nuxt

npm i https://pkg.pr.new/@clerk/nuxt@8168

@clerk/react

npm i https://pkg.pr.new/@clerk/react@8168

@clerk/react-router

npm i https://pkg.pr.new/@clerk/react-router@8168

@clerk/shared

npm i https://pkg.pr.new/@clerk/shared@8168

@clerk/tanstack-react-start

npm i https://pkg.pr.new/@clerk/tanstack-react-start@8168

@clerk/testing

npm i https://pkg.pr.new/@clerk/testing@8168

@clerk/ui

npm i https://pkg.pr.new/@clerk/ui@8168

@clerk/upgrade

npm i https://pkg.pr.new/@clerk/upgrade@8168

@clerk/vue

npm i https://pkg.pr.new/@clerk/vue@8168

commit: c51a2c3

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Mar 26, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Repository YAML (base), Organization UI (inherited)

Review profile: ASSERTIVE

Plan: Pro

Run ID: ccb2c58f-16af-41c3-9a85-83dfc97a03ae

📥 Commits

Reviewing files that changed from the base of the PR and between 118999c and c51a2c3.

📒 Files selected for processing (3)
  • .changeset/warm-turbo-forks.md
  • .github/actions/init-blacksmith/action.yml
  • .github/actions/init/action.yml

📝 Walkthrough

Walkthrough

The changes modify two GitHub Actions configuration files and add a Changesets metadata file. The turbo-team and turbo-token inputs in both .github/actions/init-blacksmith/action.yml and .github/actions/init/action.yml are changed from required to optional. Runtime validation logic in both actions is updated to accumulate missing credential fields and emit warnings instead of throwing errors, with Turbo cache environment variables skipped from export when credentials are incomplete. A new Changesets metadata file with empty frontmatter is added.

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately describes the main change: making Turbo credential inputs optional and converting validation failures into warnings for fork CI runs.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.


Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link
Member

@Ephem Ephem left a comment

Choose a reason for hiding this comment

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

Thanks! 🙏

@jacekradko jacekradko merged commit cbf1eac into main Mar 26, 2026
43 of 44 checks passed
@jacekradko jacekradko deleted the jacek/fix-turbo-env-validation branch March 26, 2026 12:54
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.

2 participants