doc: update WORKING_GROUPS.md to include Intl#3251
doc: update WORKING_GROUPS.md to include Intl#3251srl295 wants to merge 1 commit intonodejs:masterfrom
Conversation
* Intl was missing
|
rubber stamp LGTM. |
|
LGTM |
* Intl was missing Reviewed-By: Roman Reiss <me@silverwind.io> Reviewed-By: James M Snell <jasnell@gmail.com> PR-URL: #3251
|
Landed in a441aa6 |
|
consider for LTS /cc @jasnell |
|
Not a critical need for this |
|
agree, not critical for LTS .. but perhaps some such documentation should say "check the repo for the latest.." ? What if someone tried to follow a 17-and-a-half-month-old |
|
Are docs like this even included in releases (other than built from source)? Seems like API doc changes are the important ones to backport. Files like this seem more relevant to working with the live project on GitHub. |
|
considering we have been landing similar doc fixes on lts I'm going to add this to lts-watch If no one has objections within 24 hours I'll land it into v4.x-staging |
* Intl was missing Reviewed-By: Roman Reiss <me@silverwind.io> Reviewed-By: James M Snell <jasnell@gmail.com> PR-URL: #3251
* Intl was missing Reviewed-By: Roman Reiss <me@silverwind.io> Reviewed-By: James M Snell <jasnell@gmail.com> PR-URL: #3251
* Intl was missing Reviewed-By: Roman Reiss <me@silverwind.io> Reviewed-By: James M Snell <jasnell@gmail.com> PR-URL: #3251
Fixes: #3246