Skip to content
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

Feature: add telemetry to pizza generate config command flows #187

Closed
jpmcb opened this issue Sep 12, 2024 · 2 comments · Fixed by #188
Closed

Feature: add telemetry to pizza generate config command flows #187

jpmcb opened this issue Sep 12, 2024 · 2 comments · Fixed by #188

Comments

@jpmcb
Copy link
Member

jpmcb commented Sep 12, 2024

Suggested solution

The pizza generate config command flows are missing telemetry. We'll want to add it to the command options (like elsewhere):

type Options struct {
	// the path to the git repository on disk to generate a codeowners file for
	path string

	// where the '.sauced.yaml' file will go
	outputPath string

	// whether to use interactive mode
	isInteractive bool

	// from global config
	ttyDisabled bool

+	// telemetry for capturing CLI events via PostHog
+	telemetry *utils.PosthogCliClient
}

and then add capture events for this flow (captureConfigGen and capture ConfigGenFailure are probably enough)

Copy link
Contributor

open-sauced bot commented Sep 16, 2024

🎉 This issue has been resolved in version 1.5.0-beta.3 🎉

The release is available on:

Your semantic-release bot 📦🚀

Copy link
Contributor

open-sauced bot commented Sep 16, 2024

🎉 This issue has been resolved in version 1.5.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

@open-sauced open-sauced bot added the released label Sep 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants