-
Notifications
You must be signed in to change notification settings - Fork 443
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(cli): Improve telemetry #895
Conversation
57ba36d
to
2c3faf7
Compare
7d7680c
to
1c6e1cb
Compare
e0b0ecc
to
b5323a1
Compare
445ca3f
to
3c8613f
Compare
I'm working on an internal RFC and external messaging to explain what data we're collecting and what our privacy controls are, so let's hold off on merging this until that's compelte |
As prompted by the suggestion @DanielMSchmidt made on the internal RFC, I think we should stop sending |
3c8613f
to
f5575ae
Compare
3f3995d
to
a25de67
Compare
This will help us make informed decision and gives us more context on how the CDK is used Closes #826
d11d22c
to
45f00cc
Compare
45f00cc
to
c24712a
Compare
c24712a
to
cfd0f98
Compare
cfd0f98
to
6a7a193
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.
The internal RFC has been approved and this is good to go.
7a8f988
to
136c3aa
Compare
I'm going to lock this pull request because it has been closed for 30 days. This helps our maintainers find and focus on the active issues. If you've found a problem that seems related to this change, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further. |
This will help us make informed decision and gives us more context on how the CDK is used
Closes #826