doc: update lts description in the collaborator guide #3668
Conversation
jasnell
added
doc
| documentation updates, and certain performance improvements that can be | ||
| demonstrated to not break existing applications. Semver-minor changes are only | ||
| permitted if required for bug fixes and then only on a case-by-case basis with | ||
| LTS WG and possibly TSC review. Semver-major changes are only permitted if |
There was a problem hiding this comment.
Sigh. I'm gonna start just using random letters instead
|
@trevnorris ... updated. PTAL |
|
@nodejs/lts ... ping |
|
|
||
| Once a stable branch moves into Maintenance mode, only **critical** bugs, **critical** security fixes, | ||
| and documentation updates will be permitted. | ||
| Once a stable branch enters LTS, no new features may be added to that release. |
There was a problem hiding this comment.
Is that true? It seems that it's a bit more subtle than that. See #3609 for instance. It seems that the text below:
Semver-minor changes are only permitted if required for bug fixes and then only on a case-by-case basis with LTS WG and possibly Core Technical Committee (CTC) review.
actually describes this subtle situation better, and that that phrase might actually not be needed.
There was a problem hiding this comment.
Yep. Missed that one :-) Will fix!
|
@jasnell Added some comments. |
|
@misterdjules ... updated to fix the first comment (#3668 (comment)) |
|
@jasnell LGTM, thank you! |
PR-URL: #3668 Reviewed-By: Julien Gilli <jgilli@fastmail.fm>
|
Landed in dac387e |
PR-URL: #3668 Reviewed-By: Julien Gilli <jgilli@fastmail.fm>
PR-URL: #3668 Reviewed-By: Julien Gilli <jgilli@fastmail.fm>
|
landed in v4.x-staging in 9970b76 |
PR-URL: #3668 Reviewed-By: Julien Gilli <jgilli@fastmail.fm>
PR-URL: #3668 Reviewed-By: Julien Gilli <jgilli@fastmail.fm>
PR-URL: #3668 Reviewed-By: Julien Gilli <jgilli@fastmail.fm>
Update description of the LTS process in the collaborators guide based on the
current practice. @nodejs/lts