doc: update lts description in the collaborator guide#3668
doc: update lts description in the collaborator guide#3668jasnell wants to merge 1 commit intonodejs:masterfrom
Conversation
jasnell
added
doc
COLLABORATOR_GUIDE.md
Outdated
There was a problem hiding this comment.
Sigh. I'm gonna start just using random letters instead
|
@trevnorris ... updated. PTAL |
|
@nodejs/lts ... ping |
COLLABORATOR_GUIDE.md
Outdated
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>
4 participants
Update description of the LTS process in the collaborators guide based on the
current practice. @nodejs/lts