doc: document process for sharing project news - #47189
Conversation
Sorry, something went wrong.
Fixes: nodejs/TSC#1333 Signed-off-by: Michael Dawson <mdawson@devrus.com>
|
Review requested:
|
Sorry, something went wrong.
Co-authored-by: mscdex <mscdex@users.noreply.github.com>
Co-authored-by: Gireesh Punathil <gpunathi@in.ibm.com>
Co-authored-by: mscdex <mscdex@users.noreply.github.com>
Co-authored-by: mscdex <mscdex@users.noreply.github.com>
Co-authored-by: mscdex <mscdex@users.noreply.github.com>
|
|
||
| The project's approach is as follows: | ||
|
|
||
| * The project will create one "What's new in YYYY" issue in the `nodejs/node` |
There was a problem hiding this comment.
An issue seems like the wrong tool for the job. Would a GitHub discussion item be better?
Sorry, something went wrong.
There was a problem hiding this comment.
I'm if ok if a discussion item is better for the core repo. Updating
Sorry, something went wrong.
Co-authored-by: Rich Trott <rtrott@gmail.com>
|
@Trott believe I've addressed your comments/suggestions. |
Sorry, something went wrong.
| The project's approach is as follows: | ||
|
|
||
| * The project will create one "What's new in YYYY" discussion item in | ||
| the `nodejs/node` repo for news related to Node.js core. |
There was a problem hiding this comment.
I think it might be better to use a separate repo for these news. In this way, publications could subscribe to the notifications on that repository to receive the updates.
Sorry, something went wrong.
There was a problem hiding this comment.
That could make it easier for publications but the goal is to make it easy for the teams and the project both in terms of workflow and discoverability. In the case of the node-api team looking for an issue in a different repo would be more work due to the way we use a milestone to review work each week.
Even if there was a separate repo the ask was not that a new issue be created for each piece of news, only update an existing issue/discussion so once the publication subscribes to an issue I don't think it will be any different in respect to notifications.
Therefore I think using an issue/discussion item in the existing repos is better.
Sorry, something went wrong.
There was a problem hiding this comment.
ok!!
Sorry, something went wrong.
Fixes: nodejs/TSC#1333 Signed-off-by: Michael Dawson <mdawson@devrus.com> PR-URL: #47189 Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com> Reviewed-By: Chengzhong Wu <legendecas@gmail.com> Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Fixes: nodejs/TSC#1333 Signed-off-by: Michael Dawson <mdawson@devrus.com> PR-URL: #47189 Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com> Reviewed-By: Chengzhong Wu <legendecas@gmail.com> Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Fixes: nodejs/TSC#1333 Signed-off-by: Michael Dawson <mdawson@devrus.com> PR-URL: #47189 Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com> Reviewed-By: Chengzhong Wu <legendecas@gmail.com> Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Fixes: nodejs/TSC#1333 Signed-off-by: Michael Dawson <mdawson@devrus.com> PR-URL: #47189 Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com> Reviewed-By: Chengzhong Wu <legendecas@gmail.com> Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Fixes: nodejs/TSC#1333 Signed-off-by: Michael Dawson <mdawson@devrus.com> PR-URL: #47189 Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com> Reviewed-By: Chengzhong Wu <legendecas@gmail.com> Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Fixes: nodejs/TSC#1333