-
Notifications
You must be signed in to change notification settings - Fork 1.8k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat(turbo-ignore): usage metrics #7897
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
8 Ignored Deployments
|
f6255e7
to
433665a
Compare
New dependencies detected. Learn more about Socket for GitHub ↗︎
|
@@ -24,7 +24,7 @@ | |||
}, | |||
"dependencies": { | |||
"chalk": "4.1.2", | |||
"commander": "^10.0.0", | |||
"commander": "^11.0.0", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Updating to use same version across all our CLIs. Nothing breaking is relevant for us:
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Wish pnpm had workspace-wide package version definitions like in cargo...
🟢 Turbopack Benchmark CI successful 🟢Thanks |
✅ This change can build |
🟢 CI successful 🟢Thanks |
433665a
to
7317d13
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Since we are adding to a new product here is proof it respects tracking env vars for posterity https://github.com/vercel/turbo/blob/583bb0934a2521b7549380f9c49b34565804b5b1/packages/turbo-telemetry/src/config.ts#L54
Description
Anonymous usage metrics for turbo-ignore. Will help inform future roadmap for the tool.