Skip to content

Conversation

@BrennanConroy
Copy link
Member

I don't know if this is automatically applied to fabricbot or if someone needs to manually update it and this is just a record of what the bot does.

@BrennanConroy BrennanConroy requested review from a team and wtgodbe as code owners August 31, 2023 17:58
@ghost ghost added the area-infrastructure Includes: MSBuild projects/targets, build scripts, CI, Installers and shared framework label Aug 31, 2023
@ghost
Copy link

ghost commented Aug 31, 2023

Hey @dotnet/aspnet-build, looks like this PR is something you want to take a look at.

@wtgodbe
Copy link
Member

wtgodbe commented Aug 31, 2023

Doing it manually is right

@wtgodbe
Copy link
Member

wtgodbe commented Aug 31, 2023

dotnet-issue-labeler is the one that trains itself

@amcasey
Copy link
Member

amcasey commented Aug 31, 2023

Interesting. This seems like it would be more reliable than the AI one. If we think this is sufficient, I believe we can opt out of automatic PR labelling, since the PR model is separate from the Issue model.

@wtgodbe
Copy link
Member

wtgodbe commented Aug 31, 2023

I think at the moment there are some things that the AI bot labels that aren't covered by fabricbot, and there's definitely functionality in fabricbot that isn't handled by the AI bot. Unless it's causing anybody any undue harm I think it's fine to keep both

@BrennanConroy BrennanConroy merged commit ae283ea into main Aug 31, 2023
@BrennanConroy BrennanConroy deleted the BrennanConroy-patch-2 branch August 31, 2023 20:10
@ghost ghost added this to the 9.0-preview1 milestone Aug 31, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area-infrastructure Includes: MSBuild projects/targets, build scripts, CI, Installers and shared framework

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants