-
Notifications
You must be signed in to change notification settings - Fork 533
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
[GitHub] Keep GH issue templates up to date. #8942
Conversation
- net8.0-android | ||
- net9.0-android |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could these just say ".NET 8" and ".NET 9"?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I went with this because the .NET version you install isn't necessarily the bits you are using. That is, if you install .NET 9 and target net8.0-android
, you are actually using the .NET 8 bits but customers don't know that. I thought it might be more helpful for us to know what TF they are actually using.
That said, I don't have a strong opinion either way. 🤷
description: Please copy and paste any (short!) relevant log output. Longer logs can be attached as .txt files. This is likely a [diagnostic MSBuild log](https://docs.microsoft.com/en-us/xamarin/android/troubleshooting/troubleshooting#diagnostic-msbuild-output) or attach a [MSBuild binlog](https://docs.microsoft.com/en-us/visualstudio/msbuild/obtaining-build-logs-with-msbuild?view=vs-2022#save-a-binary-log). | ||
description: Please copy and paste any (short!) relevant log output. Longer logs can be attached as .txt files. This is likely a [diagnostic MSBuild log](https://docs.microsoft.com/en-us/xamarin/android/troubleshooting/troubleshooting#diagnostic-msbuild-output) or attach a [MSBuild binlog](https://aka.ms/binlog). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If there are docs.microsoft.com
URLs, should those be learn.microsoft.com
now?
I think we are also supposed to remove the /en-us/
, so it will localize for other languages.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Correct on both counts: use learn.microsoft.com, and remove /en-us
from the URLs.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed
placeholder: E.g. VS 2022 17.8.0, VSMac 17.6.7, .NET 8.0.100, etc. | ||
placeholder: E.g. VS 2022 17.9.0, .NET 8.0.100, etc. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should we ask for the VS Code version? Maybe just the version of the MAUI extension?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I believe the intent here is to help us determine what version of our code they are using. Since VSCode doesn't ship our bits, I don't know if its version helps any. For VSCode users I suspect we would rather they provide the .NET version.
* main: [GitHub] Keep GH issue templates up to date. (#8942)
Update GH issue templates, removing EOL Classic and .NET 7.