-
Notifications
You must be signed in to change notification settings - Fork 896
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
Clarify whether unreleased changes should be implemented or not #2728
Comments
@open-telemetry/specs-approvers what do you think about this? |
I think the process does not really suggest that releases are a potential point of re-discussion or even sign-off of changes? |
In JS we commonly implement unreleased changes, especially when we have opened an issue and something is clarified in response to our issue. Waiting for the release may slow down a process that some already see as too slow. I would suggest this policy should only apply to stable released packages. |
Here is what I would like to propose:
I think this is a good tradeoff. We retain flexibility to fix mistakes, allow implementations to work faster, but still have mechanisms in place to catch spec mistakes and avoid making such mistakes part of stable implementation releases. Thoughts? |
Otel spec is released monthly. Between the release we submit PRs and make changes to the spec repo, accumulating unreleased changes.
Occasionally changes are disputed after merging, with the possibility of reverting the changes before a release is made, see for example #2726
Question: should the unreleased changes considered to be official part of the spec or they only become part of the spec once the release is made?
The text was updated successfully, but these errors were encountered: