Skip to content

Rename area-runtime in fabricbot #50460

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

Merged
merged 1 commit into from
Aug 31, 2023
Merged

Rename area-runtime in fabricbot #50460

merged 1 commit into from
Aug 31, 2023

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 wtgodbe and a team 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.

3 participants