Updated Initialize.cs to show modern dotnet4+ versions#6128
Conversation
valadas
left a comment
There was a problem hiding this comment.
Except the build failed...
Removed untested line of code, restored extra newline after close brace `}`. Build works again for me locally.
Sorry about that. Fixed and works for me now (local build). |
|
There is still 2:
|
Are you messing with me? I just fixed the opposite above, "SA1513 - Closing brace should be followed by blank line" I tried this in both VS Code and VS 2022. The only way not to get one of the two errors is ... Oh, so both editors are just inserting whitespace up to the indented cursor. I am supposed to know to spot that and fix it? Okay, sorry, lesson learned. Follow the rules even if you can't see the difference. I think this will fix it, at least I get neither error in VS 2022 when I force the blank lines. |
Replaced whitespace with blank lines to avoid SA1028 and SA1513 simultaneously. :)
Changed to `return` to match first with releases in descending order
jeremy-farrance
left a comment
There was a problem hiding this comment.
Okay, code is updated.
Fixes #6127
Summary
Quick update so that the .NET Framework version shown (e.g. in the PersonaBar) is accurate up to v4.8.1.
Also updated the code style (no more
else) making it easy to added new versions in the future.Reference: DotNet Versions via Code