-
Notifications
You must be signed in to change notification settings - Fork 5.3k
Fixes based on feedback for the original #59929 #59967
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
Fixes based on feedback for the original #59929 #59967
Conversation
|
Tagging subscribers to this area: @vitek-karas, @agocke, @VSadov Issue DetailsAdds apphost_version to the URL (most important fix) Cleans up some stuff in tests
|
Co-authored-by: Elinor Fung <[email protected]>
|
CI failure is #59908 |
agocke
left a comment
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.
LGTM
Co-authored-by: Elinor Fung <[email protected]> (cherry picked from commit 3d171f9)
…ssing (#60045) * Show dialog when running .NET 6 GUI single-file on 5.0 hostfxr (#59929) Show dialog when running .NET 6 GUI single-file on 5.0 hostfxr (cherry picked from commit 2f76c08) * Fixes based on feedback for the original #59929 (#59967) Co-authored-by: Elinor Fung <[email protected]> (cherry picked from commit 3d171f9) Co-authored-by: Vitek Karas <[email protected]>
Adds apphost_version to the URL (most important fix)
Adds version info to the error message for 6.0
Cleans up some stuff in tests