Skip to content

Conversation

@daxian-dbw
Copy link
Member

PR Summary

Use Needs-Triage instead of Issue-Question for new issues.

PR Checklist

@iSazonov
Copy link
Collaborator

I suggest to set both Needs-Triage and Issue-Question labels for new issues. The same for Issue-Enhancement - add Needs-Triage too.

@daxian-dbw
Copy link
Member Author

Good call on adding Needs-Triage along with Issue-Enhancement for feature request issues.
For bug issues, does it make sense to add Issue-Question automatically? In my personal opinion, Issue-Question should be applied to those issues that are really just a question, and thus when applying this label, it means the issue has already been triaged.

@iSazonov
Copy link
Collaborator

iSazonov commented Jan 21, 2021

Good call on adding Needs-Triage along with Issue-Enhancement for feature request issues.
For bug issues, does it make sense to add Issue-Question automatically? In my personal opinion, Issue-Question should be applied to those issues that are really just a question, and thus when applying this label, it means the issue has already been triaged.

In the case we should do the some for Issue-Enhancement :-)

Currently I set Issue-Question for all issues as a start point. New process is still not clear for me - who and how should process Needs-Triage? We could automatically set only Needs-Triage and after triage (or I could does this independently leaving removing Needs-Triage to WGs) manually set Issue-Question for minor issues, Issue-Bug for bugs and Issue-Enhancement for new functionality.

@daxian-dbw
Copy link
Member Author

Issue-Enhancement is auto-set for issues that are opened using the Feature_Request.md template. I think that's explicit enough.

who and how should process

According to the WG doc, the corresponding WG should triage the issue, and Needs-Triage label should be removed after the triage.

@iSazonov
Copy link
Collaborator

Issue-Enhancement is auto-set for issues that are opened using the Feature_Request.md template. I think that's explicit enough.

Currently I have to decrease most of such issues to Issue-Question level because they are poorly justified.

According to the WG doc, the corresponding WG should triage the issue, and Needs-Triage label should be removed after the triage.

I would be happy if, from tomorrow, the working group members (who are now exclusively MSFT employees) have been doing this work on a daily basis. But I strongly doubt that it will be so tomorrow. Should I continue this work?

@daxian-dbw
Copy link
Member Author

daxian-dbw commented Jan 25, 2021

I would be happy if, from tomorrow, the working group members (who are now exclusively MSFT employees) have been doing this work on a daily basis. But I strongly doubt that it will be so tomorrow. Should I continue this work?

I'm going to do my WG duty starting from today 😄 I strongly believe you should be part of some of the WGs, given that you are a maintainer and also you have been doing the WG work for such a long time!
I left a comment in the WG PR https://github.com/PowerShell/PowerShell/pull/14603/files#r564015516

@daxian-dbw
Copy link
Member Author

@iSazonov I will merge this PR for now. Let's continue the process discussion in the WG PR. We will see how it works.

@daxian-dbw daxian-dbw closed this Jan 25, 2021
@daxian-dbw daxian-dbw reopened this Jan 25, 2021
@daxian-dbw
Copy link
Member Author

@rjmholt It seems something is wrong with the CI 😕 Maybe just forcefully merge this one?

@TravisEz13
Copy link
Member

nothing is wrong with CI, just doesn't run on files in this folder.

@daxian-dbw
Copy link
Member Author

That's good to know. Then I will just merge this PR.

@daxian-dbw daxian-dbw merged commit 7e9118c into PowerShell:master Jan 26, 2021
@daxian-dbw daxian-dbw deleted the bug-temp branch January 26, 2021 01:50
@daxian-dbw daxian-dbw added the CL-NotInBuild Indicates that a PR is reverted and not part of the build. label Jan 26, 2021
@daxian-dbw daxian-dbw added this to the 7.2.0-preview.3 milestone Jan 26, 2021
@iSazonov
Copy link
Collaborator

iSazonov commented Jan 26, 2021

Maybe set Needs-Triage label on all opened issues in the repo by script?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CL-NotInBuild Indicates that a PR is reverted and not part of the build.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants