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.

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

[FabricBot] Apply 'needs-triage' label to new issues. #7327

Merged
merged 2 commits into from
Sep 1, 2022
Merged

Conversation

jpobst
Copy link
Contributor

@jpobst jpobst commented Sep 1, 2022

Today we rely on our templates to apply the needs-triage label to new issues. However, there are several ways issues can be added to our repo that bypasses our templates:

  • Using the "blank" template
  • Moved from another repository
  • Moved from vsfeedback

FabricBot can ensure every new issue is tagged with needs-triage.

Note that the first commit simply allows the FabricBot UI to reformat our existing config file to its desired format. It does not make any changes.

To see what is actually being added, look at the second commit.

Comment on lines 572 to 578
"actions": [
{
"name": "addLabel",
"parameters": {
"label": "needs-triage"
}
}
Copy link
Member

Choose a reason for hiding this comment

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

You can't add a label twice, right? So it's fine if issues come in with the label already?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I would assume that is true. You cannot add the label twice in the GitHub UI at least.

@jpobst jpobst merged commit a0377cd into main Sep 1, 2022
@jpobst jpobst deleted the fabric-bot branch September 1, 2022 20:26
grendello added a commit to grendello/xamarin-android that referenced this pull request Sep 2, 2022
* main:
  Bump to dotnet/installer@2d1a4de 7.0.100-rc.2.22426.5 (dotnet#7314)
  [FabricBot] Apply 'needs-triage' label to new issues. (dotnet#7327)
  [Mono.Android] .NET 6+ & GetHttpMessageHandler & null, oh my! (dotnet#7214)
grendello added a commit to grendello/xamarin-android that referenced this pull request Sep 2, 2022
* mm-runtime:
  Bump to dotnet/installer@2d1a4de 7.0.100-rc.2.22426.5 (dotnet#7314)
  [FabricBot] Apply 'needs-triage' label to new issues. (dotnet#7327)
  [Mono.Android] .NET 6+ & GetHttpMessageHandler & null, oh my! (dotnet#7214)
@github-actions github-actions bot locked and limited conversation to collaborators Jan 24, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants