Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

doc: feature management proposal #41420

Closed
wants to merge 4 commits into from
Closed

doc: feature management proposal #41420

wants to merge 4 commits into from

Conversation

mhdawson
Copy link
Member

@mhdawson mhdawson commented Jan 6, 2022

Refs: #40823
Refs: #41113

Signed-off-by: Michael Dawson mdawson@devrus.com

@nodejs-github-bot nodejs-github-bot added the doc Issues and PRs related to the documentations. label Jan 6, 2022
VoltrexKeyva
VoltrexKeyva reviewed Jan 6, 2022
View reviewed changes
doc/guides/feature-request-management.md Outdated Show resolved Hide resolved
doc/guides/feature-request-management.md Outdated Show resolved Hide resolved
doc/guides/feature-request-management.md Outdated Show resolved Hide resolved
@Trott
Copy link
Member

Trott commented Jan 7, 2022

If we're going to add a doc like this, we'll probably want to include a link to it in https://github.com/nodejs/node/blob/master/.github/ISSUE_TEMPLATE/2-feature-request.yml.

@mhdawson
Copy link
Member Author

mhdawson commented Jan 7, 2022

@mscdex @bnb @aduh95 @Trott thanks for all the comments. I think I've addressed them all.

aduh95
aduh95 reviewed Jan 7, 2022
View reviewed changes
doc/guides/feature-request-management.md Outdated Show resolved Hide resolved
doc/guides/feature-request-management.md Outdated Show resolved Hide resolved
@mhdawson mhdawson mentioned this pull request Jan 13, 2022
Closed
jasnell
jasnell reviewed Jan 17, 2022
View reviewed changes
doc/guides/feature-request-management.md Outdated

## Automated feature request handling

Our experience is that most feature requests that are
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why are these line lengths so short? :-P

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I prefer that to having to check each one for being too long :)

@Mesteery
Copy link
Contributor

Should this be moved to doc/contributing instead of doc/guides?

@mhdawson
Copy link
Member Author

@Mesteery not that the other PR has landed this does need a "rebase"

@mhdawson
doc: feature management proposal
142caa2 
Refs: nodejs#40823
Refs: nodejs#41113

Signed-off-by: Michael Dawson <mdawson@devrus.com>
@mhdawson
Copy link
Member Author

@Mesteery moved.

@mhdawson
Copy link
Member Author

@targos just wondering if you'd had a chance to consider what updates we might make to this in order to incorporate your suggestion about using https://github.com/orgs/nodejs/projects/4

@targos
Copy link
Member

targos commented Jan 29, 2022

@mhdawson I think this PR doesn't have to wait for this project. I'm convinced that it will be useful but I need to discuss with other people before documenting it.

@mhdawson
Copy link
Member Author

@targos thanks. @nodejs/tsc it would be good to have a few more approvals since once this lands I'll work to turn on stale bot and we'll have some issues tagged feature being closed as outline in the process.

@mhdawson @Trott
Update .github/ISSUE_TEMPLATE/2-feature-request.yml
467d1b6 
Co-authored-by: Rich Trott <rtrott@gmail.com>
RaisinTen
RaisinTen approved these changes Feb 1, 2022
View reviewed changes
Copy link
Contributor

@RaisinTen RaisinTen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nit: HEAD links :)

doc/contributing/feature-request-management.md Outdated Show resolved Hide resolved
doc/contributing/feature-request-management.md Outdated Show resolved Hide resolved
mhdawson and others added 2 commits February 1, 2022 17:34
@mhdawson @RaisinTen
Update doc/contributing/feature-request-management.md
a378978 
Co-authored-by: Darshan Sen <raisinten@gmail.com>
@mhdawson @RaisinTen
Update doc/contributing/feature-request-management.md
da1e6a4 
Co-authored-by: Darshan Sen <raisinten@gmail.com>
mcollina
mcollina approved these changes Feb 2, 2022
View reviewed changes
Copy link
Member

@mcollina mcollina left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@mhdawson
Copy link
Member Author

mhdawson commented Feb 3, 2022 •
edited
Loading

Agreed in last weeks TSC meeting on actions, does still need to be on agenda? - Wrong PR

mhdawson added a commit that referenced this pull request Feb 4, 2022
@mhdawson
doc: feature management proposal
6c0eb94 
Refs: #40823
Refs: #41113

Signed-off-by: Michael Dawson <mdawson@devrus.com>

PR-URL: #41420
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Reviewed-By: Mary Marchini <oss@mmarchini.me>
Reviewed-By: Darshan Sen <raisinten@gmail.com>
Reviewed-By: Matteo Collina <matteo.collina@gmail.com>
Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com>
@mhdawson
Copy link
Member Author

mhdawson commented Feb 4, 2022

Landed in 6c0eb94

@mhdawson mhdawson closed this Feb 4, 2022
ruyadorno pushed a commit that referenced this pull request Feb 8, 2022
@mhdawson @ruyadorno
doc: feature management proposal
451f333 
Refs: #40823
Refs: #41113

Signed-off-by: Michael Dawson <mdawson@devrus.com>

PR-URL: #41420
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Reviewed-By: Mary Marchini <oss@mmarchini.me>
Reviewed-By: Darshan Sen <raisinten@gmail.com>
Reviewed-By: Matteo Collina <matteo.collina@gmail.com>
Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com>
@ruyadorno ruyadorno mentioned this pull request Feb 8, 2022
Merged
danielleadams pushed a commit that referenced this pull request Mar 2, 2022
@mhdawson @danielleadams
doc: feature management proposal
1334f61 
Refs: #40823
Refs: #41113

Signed-off-by: Michael Dawson <mdawson@devrus.com>

PR-URL: #41420
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Reviewed-By: Mary Marchini <oss@mmarchini.me>
Reviewed-By: Darshan Sen <raisinten@gmail.com>
Reviewed-By: Matteo Collina <matteo.collina@gmail.com>
Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com>
@danielleadams danielleadams mentioned this pull request Mar 3, 2022
Merged
danielleadams pushed a commit that referenced this pull request Mar 3, 2022
@mhdawson @danielleadams
doc: feature management proposal
8567aff 
Refs: #40823
Refs: #41113

Signed-off-by: Michael Dawson <mdawson@devrus.com>

PR-URL: #41420
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Reviewed-By: Mary Marchini <oss@mmarchini.me>
Reviewed-By: Darshan Sen <raisinten@gmail.com>
Reviewed-By: Matteo Collina <matteo.collina@gmail.com>
Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com>
danielleadams pushed a commit to danielleadams/node that referenced this pull request Mar 4, 2022
@mhdawson @danielleadams
doc: feature management proposal
b3134db 
Refs: nodejs#40823
Refs: nodejs#41113

Signed-off-by: Michael Dawson <mdawson@devrus.com>

PR-URL: nodejs#41420
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Reviewed-By: Mary Marchini <oss@mmarchini.me>
Reviewed-By: Darshan Sen <raisinten@gmail.com>
Reviewed-By: Matteo Collina <matteo.collina@gmail.com>
Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com>
danielleadams pushed a commit to danielleadams/node that referenced this pull request Mar 4, 2022
@mhdawson @danielleadams
doc: feature management proposal
6be94bf 
Refs: nodejs#40823
Refs: nodejs#41113

Signed-off-by: Michael Dawson <mdawson@devrus.com>

PR-URL: nodejs#41420
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Reviewed-By: Mary Marchini <oss@mmarchini.me>
Reviewed-By: Darshan Sen <raisinten@gmail.com>
Reviewed-By: Matteo Collina <matteo.collina@gmail.com>
Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com>
danielleadams pushed a commit that referenced this pull request Mar 8, 2022
@mhdawson @danielleadams
doc: feature management proposal
4472527 
Refs: #40823
Refs: #41113

Signed-off-by: Michael Dawson <mdawson@devrus.com>

PR-URL: #41420
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Reviewed-By: Mary Marchini <oss@mmarchini.me>
Reviewed-By: Darshan Sen <raisinten@gmail.com>
Reviewed-By: Matteo Collina <matteo.collina@gmail.com>
Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com>
danielleadams pushed a commit that referenced this pull request Mar 14, 2022
@mhdawson @danielleadams
doc: feature management proposal
c742a1c 
Refs: #40823
Refs: #41113

Signed-off-by: Michael Dawson <mdawson@devrus.com>

PR-URL: #41420
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Reviewed-By: Mary Marchini <oss@mmarchini.me>
Reviewed-By: Darshan Sen <raisinten@gmail.com>
Reviewed-By: Matteo Collina <matteo.collina@gmail.com>
Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@VoltrexKeyva VoltrexKeyva VoltrexKeyva left review comments

@Trott Trott Trott left review comments

@bnb bnb bnb left review comments

@mscdex mscdex mscdex left review comments

@RaisinTen RaisinTen RaisinTen approved these changes

@aduh95 aduh95 aduh95 approved these changes

@gireeshpunathil gireeshpunathil gireeshpunathil approved these changes

@mmarchini mmarchini mmarchini approved these changes

@jasnell jasnell jasnell approved these changes

@mcollina mcollina mcollina approved these changes

Assignees
No one assigned
Labels
doc Issues and PRs related to the documentations.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.