Skip to content

Improved Bot Response for investigation issue #45254

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 2 commits into from
Nov 24, 2022
Merged

Conversation

mkArtakMSFT
Copy link
Contributor

No description provided.

@mkArtakMSFT mkArtakMSFT added the area-infrastructure Includes: MSBuild projects/targets, build scripts, CI, Installers and shared framework label Nov 23, 2022
@mkArtakMSFT mkArtakMSFT requested review from a team, dougbu and wtgodbe as code owners November 23, 2022 21:20
@ghost
Copy link

ghost commented Nov 23, 2022

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

@mkArtakMSFT
Copy link
Contributor Author

@seangwright, thanks for your feedback here: #44857 (comment)

Do you think this would be a better response by the Bot?

@seangwright
Copy link
Contributor

Yes! This is definitely better.

Maybe this is more about process for the aspnetcore repo vs the messaging for this bot response, but seeing the .NET 8 Planning label get assigned has me immediately thinking "Oh, so this won't be fixed for .NET 7", which could cause confusion and frustration if a developer is reporting a show-stopper issue.

I'm guessing .NET 8 Planning is used to represent "these are things we are working on now vs. postponed for future"? If that's true, maybe some clarity there could help.

@mkArtakMSFT
Copy link
Contributor Author

You're right, the .NET 8 Planning milestone is simply a bucket of work to be considered for .NET 8. Depending on the feedback we will receive over time on some of these issues, we will choose to also patch these for down-level releases too. We've tried our best to clarify that in our Triage Process document. Unfortunately, not everyone reads these and we found no better way so far to address this.

@seangwright
Copy link
Contributor

seangwright commented Nov 23, 2022

@mkArtakMSFT Maybe moving the triage process link earlier in the message (maybe at the top?) would get more people to click it.

I honestly never saw the link before even though I've seen this bot message many times here!

See below as an attempt to bring more focus to the link:


Thanks for helping ASP.NET Core.

To learn more about what this message means, what to expect next, and how this issue will be handled you can read more about our triage process.

We're moving this issue to the .NET 8 Planning milestone for future evaluation / consideration. Because it's not immediately obvious what is causing this behavior, we would like to keep this around to collect more feedback, which can later help us determine how to handle this. We will re-evaluate this issue, during our next planning meeting(s).
If we later determine, that the issue has no community involvement, or it's very rare and low-impact issue, we will close it - so that the team can focus on more important and high impact issues.

@mkArtakMSFT
Copy link
Contributor Author

mkArtakMSFT commented Nov 23, 2022

I like it and we can sure try to go with that message. Will update the PR soon.
Not sure about the helping ASP.NET Core part, so will keep that out, for now.

@mkArtakMSFT mkArtakMSFT merged commit 5fb23e3 into main Nov 24, 2022
@mkArtakMSFT mkArtakMSFT deleted the mkArtakMSFT/botResponse branch November 24, 2022 02:47
@ghost ghost added this to the 8.0-preview1 milestone Nov 24, 2022
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