-
Notifications
You must be signed in to change notification settings - Fork 29.7k
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
Document process for Semver Major Changes and Deprecation Policy #7912
Comments
+1 from me. I think that we should have this better documented. We seem to have run into issues a few times now related to breaking changes. |
Having a documented policy would be definitely be good. We do need to make sure, however, that it includes the flexibility for us to make certain necessary semver-major changes without requiring a full deprecation cycle, however, as that would simply make things far too difficult to manage over time. As a related aside, e.g.
|
+1 to both of those — documenting the process and commit comments.
That can also result in false positives, if a «semver-major»/«semver-minor» PR includes several commits, only part of which are actually semver-major/semver-minor. E.g. #7562 was the case. |
Changed title as it appears we have documentation for deprecation policy now edit: lol nvm that was the old issue. |
Semver changes policy: Included under https://github.com/nodejs/node/blob/3702ae732eadd9d7c64681edcdd76cd7a30f6381/COLLABORATOR_GUIDE.md#accepting-modifications. Deprecation policy: https://github.com/nodejs/node/blob/3702ae732eadd9d7c64681edcdd76cd7a30f6381/COLLABORATOR_GUIDE.md#deprecations. If there's stuff missing, feel free to re-open and specify what's missing or open a new issue. Or if I'm just misguided for closing this, re-open. :-D |
It would be a good idea for us to document a policy for how Semver Major changes are landed and how deprecations are handled.
@isaacs has offered to write some copy, which I think should live in our collaborator guide
@nodejs/ctc please chime in with your thoughts on this.
The text was updated successfully, but these errors were encountered: