[v22.x backport] doc: remove obsolete Boxstarter automated install#63190
[v22.x backport] doc: remove obsolete Boxstarter automated install#63190MikeMcC399 wants to merge 1 commit intonodejs:v22.x-stagingfrom
Conversation
|
Review requested:
|
PR-URL: nodejs#61785 Fixes: nodejs#61690 Reviewed-By: Richard Lau <[email protected]> Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: Marco Ippolito <[email protected]> Reviewed-By: Stefan Stojanovic <[email protected]> Reviewed-By: Tierney Cyren <[email protected]> Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Rafael Gonzaga <[email protected]> Signed-off-by: Mike McCready <[email protected]>
9dd96ca to
73ec371
Compare
|
I have added to satisfy linting, despite what it says in https://github.com/nodejs/node/blob/main/doc/contributing/pull-requests.md#step-4-commit that it is not required for backport commits. |
|
@MikeMcC399 as per the backport rule I believe we should not change commit message also, it's fine to let if fail for now node/doc/contributing/pull-requests.md Lines 207 to 208 in e15f905 |
|
Since I was the author also of the original PR that I am backporting, and at the time the original PR was submitted and landed there was no requirement for I've added a separate issue #63192 to clarify the linting / process problem. I would only remove the |
Backports #61785 to v22.x
PR-URL: #61785
Fixes: #61690
Reviewed-By: Richard Lau [email protected]
Reviewed-By: Colin Ihrig [email protected]
Reviewed-By: Marco Ippolito [email protected]
Reviewed-By: Stefan Stojanovic [email protected]
Reviewed-By: Tierney Cyren [email protected]
Reviewed-By: Luigi Pinca [email protected]
Reviewed-By: Rafael Gonzaga [email protected]