opentelemetry-specification
opentelemetry-specification copied to clipboard
open-telemetry
Reduce entries in spec matrix to make it easier to read
The spec compliance matrix for Metrics has a lot of entries, and I feel it is too detailed. This PR is proposing to collapse some of them - specifically this PR makes a single entry for "Support Counter instrument", which is implied to indicate everything related to Counter is supported, as opposed to a line entry for each of creation/add/add-with-attribute/value-validation etc. I don't see anything important enough for a user is "lost" with this condensed version.
If I get an okay for this direction, I can continue with doing same for other type of instruments as well. Overall, it'll make it easy for all to follow the matrix, without losing anything....
This PR was marked stale due to lack of activity. It will be closed in 7 days.
![github-actions[bot] avatar](https://avatars.githubusercontent.com/in/15368?v=4 "Published by a pub.dev verified publisher"){kind=small}
@carlosalberto can you give a thumbs-up/down (as you are the assignee for this by the bot!)
This PR was marked stale due to lack of activity. It will be closed in 7 days.
Oops, sorry for missing this one. Let's poke the @open-telemetry/specs-metrics-approvers to confirm this is fine.
This PR was marked stale due to lack of activity. It will be closed in 7 days.
This PR was marked stale due to lack of activity. It will be closed in 7 days.
The more detailed view can be useful to maintainers. But since there isn't a way collapse to make it easier for a user at the same time as keeping it I wouldn't really argue it should stay for that purpose.
Thanks. Given the approvals, I'll spend some time to replicate this for other instruments as well and other places where it makes sense via small PRs.