Skip to content

chore: disable telemetry in preview builds#25

Merged
AmanVarshney01 merged 1 commit intomainfrom
aman/disable-preview-telemetry
Mar 17, 2026
Merged

chore: disable telemetry in preview builds#25
AmanVarshney01 merged 1 commit intomainfrom
aman/disable-preview-telemetry

Conversation

@AmanVarshney01
Copy link
Member

@AmanVarshney01 AmanVarshney01 commented Mar 17, 2026

Summary

  • stop injecting telemetry env vars into the PR preview build job
  • keep telemetry enabled for release builds only

Notes

  • this only changes .github/workflows/publish.yml
  • preview packages will now build without a baked PostHog key
  • release packaging remains unchanged

Summary by CodeRabbit

Chores

  • Simplified CI/CD workflow configuration by removing unused settings.

@coderabbitai
Copy link

coderabbitai bot commented Mar 17, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro

Run ID: 4b1d7362-ce5c-45d3-9722-e83707a46684

📥 Commits

Reviewing files that changed from the base of the PR and between d5f98e9 and 8360f9e.

📒 Files selected for processing (1)
  • .github/workflows/publish.yml
💤 Files with no reviewable changes (1)
  • .github/workflows/publish.yml

Walkthrough

Environment variables CREATE_PRISMA_TELEMETRY_API_KEY and CREATE_PRISMA_TELEMETRY_HOST were removed from the preview job configuration in the GitHub Actions publish workflow.

Changes

Cohort / File(s) Summary
GitHub Actions Workflow Configuration
.github/workflows/publish.yml
Removed telemetry-related environment variables (CREATE_PRISMA_TELEMETRY_API_KEY, CREATE_PRISMA_TELEMETRY_HOST) from the preview job's environment configuration.
🚥 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 clearly summarizes the main change: disabling telemetry in preview builds by removing environment variables from the preview job configuration.
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.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch aman/disable-preview-telemetry
📝 Coding Plan
  • Generate coding plan for human review comments

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

@github-actions
Copy link

PR preview published

  • Version: 0.3.0-pr.25.36.1
  • Tag: pr25
  • Run with Bun: bunx create-prisma@pr25
  • Run with npm: npx create-prisma@pr25
  • Run with Yarn: yarn dlx create-prisma@pr25
  • Run with pnpm: pnpm dlx create-prisma@pr25
  • Run with Deno: deno run -A npm:create-prisma@pr25
  • Workflow run: https://github.com/prisma/create-prisma/actions/runs/23197003360

@AmanVarshney01 AmanVarshney01 changed the title ci: disable telemetry in preview builds chore: disable telemetry in preview builds Mar 17, 2026
@AmanVarshney01 AmanVarshney01 merged commit 83ad690 into main Mar 17, 2026
3 checks passed
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