-
Notifications
You must be signed in to change notification settings - Fork 7
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
Various issues #5
Comments
Don't we want to use |
I think we agreed upon including the build slug in these analytics too. Also, pls make sure this is the format the plugin is sending (or going to send). Tooling should approve this, it might be worth showing this to Viktor too. |
It would be nice to test if we actually call the tracker with the right data. (We can make this into a maintenance card.) |
How do we test if |
TBH I'm not really sure about this endpoint. It would be nice to see a use case how this is getting used by a step. (Steps don't really do stuff ad-hoc, it might be a better approach to e.g. add some optional comment field in the metrics data.) We might want to talk about this w Viktor and tooling. |
It's a bit unexpected to have two return values here. Why we need these? |
They'd like to use this for ad-hoc logs/warnings/alerts, e.g. when a step processes Provisioning Profiles, which today can have the platforms |
Yes, definitely include both the AppSlug and the BuildSlug everywhere (when available)! |
https://github.com/bitrise-team/bitrise-step-analytics/blob/93bcd53063f6b9f626332bf71243663d3607729d/main.go#L14
Let's call this
step-analytics
The text was updated successfully, but these errors were encountered: