Skip to content

Conversation

@TravisEz13
Copy link
Member

@TravisEz13 TravisEz13 commented Jan 8, 2020

PR Summary

Add configuration for the issue label bot so when we enable it, the bot will use our labels.

PR Context

https://github.com/marketplace/issue-label-bot

PR Checklist

@TravisEz13 TravisEz13 requested a review from joeyaiello as a code owner January 8, 2020 20:28
@ghost ghost assigned TravisEz13 Jan 8, 2020
@TravisEz13
Copy link
Member Author

@RDIL
Copy link
Contributor

RDIL commented Jan 8, 2020

So far, from what I have seen, I’m not very optimistic for results here in this repository. Since the bot has the entire GitHub as it’s data set, one single person rating a deduction incorrectly throws off the entire data set by a bit. I don’t think it’s worth it, and we should most likely stick with issue templates which now have support for adding labels based on which template you use. If there is a self host option for it, that would most likely work 100x better because it will only be collecting powershell data - but the current hosted one is a no from me. Of course, this is just my opinion, but I think it should be taken under consideration.

@TravisEz13
Copy link
Member Author

@RDIL have you used it only in the docker repo? or other places?

@RDIL
Copy link
Contributor

RDIL commented Jan 8, 2020

No, I have not used it in other repositories.

@TravisEz13
Copy link
Member Author

@RDIL I understand your concern, but I don't think we have enough data to determine if it's useful from the experiment in the docker repo. We would get that data much faster from the PowerShell repo.

@RDIL
Copy link
Contributor

RDIL commented Jan 8, 2020

If you want to give it a try, you can go ahead with it.

@adityapatwardhan adityapatwardhan merged commit 667ac4b into PowerShell:master Jan 8, 2020
@daxian-dbw daxian-dbw added this to the 7.1.0-preview.1 milestone Jan 9, 2020
@TravisEz13 TravisEz13 added CL-BuildPackaging Indicates that a PR should be marked as a build or packaging change in the Change Log CL-Tools Indicates that a PR should be marked as a tools change in the Change Log and removed CL-BuildPackaging Indicates that a PR should be marked as a build or packaging change in the Change Log labels Jan 11, 2020
@TravisEz13 TravisEz13 deleted the issue_label_bot_config branch January 11, 2020 01:14
@ghost
Copy link

ghost commented Mar 26, 2020

🎉v7.1.0-preview.1 has been released which incorporates this pull request.:tada:

Handy links:

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

Labels

CL-Tools Indicates that a PR should be marked as a tools change in the Change Log

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants