-
Notifications
You must be signed in to change notification settings - Fork 897
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
Remove schema transformations from the concept of telemetry stability for now #3372
Remove schema transformations from the concept of telemetry stability for now #3372
Conversation
ae7a564
to
94016d4
Compare
Also fixes #2606 |
Co-authored-by: Liudmila Molkova <limolkova@microsoft.com>
Why remove instead of marking the individual sections as "Experimental"? |
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.
Blocking since I don't think we need to delete the text when we can instead mark the section "Experimental".
Happy to unblock this if we think the sections are wrong and do need to be removed, but that is not what the PR description says at the moment. In the past our approach about useful but not yet ready to make "Stable" sections of the spec was to label section stability individually.
some of the language was embedded into sections which we would otherwise want to mark stable I will try to re-work the PR in a way that pulls that language out into separate sections instead of deleting though let's resolve the new discussions going on in #3296 first, as that may affect the breakout as well |
closing in favor of #3380 |
Fixes #3296
Fixes #2606
Changes
Prepares
versioning-and-stability.md
andtelemetry-stability.md
documents to be markedStable
themselves, by removing schema transformations from the concept of telemetry stability for now.Note: I wasn't quite sure where to move the pieces that I removed, definitely open to feedback.