Remove dead code from SemanticVersion class#14320
Remove dead code from SemanticVersion class#14320iSazonov wants to merge 1 commit intoPowerShell:masterfrom
Conversation
|
@daxian-dbw @rjmholt Could you please explain how the Update: looks like a tricky bug in .Net Runtime. :-( |
|
@iSazonov This code path shouldn't be removed. It's the reason why you see the following from a |
|
Besides, this particular impact won't affect stable versions, because |
|
@daxian-dbw Thanks for clarify! |
I see your point but:
|
PR Summary
Remove dead code from SemanticVersion class. It was added pending the development of semantic version story but we got no feedback and no progress. Currently we are waiting .Net for the story.
The PR speeds up startup scenario on my laptop up to 50 ms.
PR Context
Related #14268
PR Checklist
.h,.cpp,.cs,.ps1and.psm1files have the correct copyright headerWIP:or[ WIP ]to the beginning of the title (theWIPbot will keep its status check atPendingwhile the prefix is present) and remove the prefix when the PR is ready.