Skip to content

Conversation

@dcantah
Copy link
Contributor

@dcantah dcantah commented Oct 21, 2021

This change adds three new definitions to the osversion package. All
three definitions are all of the 21H2 builds across Windows 10, Windows Server
and Windows 11, which all have different build numbers.

The approach taken was to prefix the definitions with Win10, WinServer and
Win11 respectively.

Signed-off-by: Daniel Canter [email protected]

@dcantah dcantah requested a review from a team as a code owner October 21, 2021 23:51
@dcantah
Copy link
Contributor Author

dcantah commented Oct 22, 2021

@kevpar Assigning you as we'd talked on ways to handle this. Now that the build number for the Windows 10 release is known, I've went with a prefix approach. Let me know if this looks sound.

@dcantah
Copy link
Contributor Author

dcantah commented Nov 8, 2021

@kevpar ptal if you have some time

This change adds three new definitions to the osversion package. All
three definitions are all of the 21H2 builds across Windows 10, Windows Server
and Windows 11, which all have different build numbers.

The approach taken was to prefix the definitions with Win10, WinServer and
Win11 respectively.

Signed-off-by: Daniel Canter <[email protected]>
@dcantah dcantah force-pushed the osversion-21h2 branch 2 times, most recently from eb41aaa to 5b0de38 Compare November 10, 2021 22:13
@dcantah
Copy link
Contributor Author

dcantah commented Nov 10, 2021

@ambarve Added your suggestion for naming, ptal

Copy link
Contributor

@ambarve ambarve left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@dcantah dcantah merged commit 3a8cd1e into microsoft:master Nov 11, 2021
dcantah added a commit to dcantah/hcsshim that referenced this pull request Jan 10, 2022
* Add 21H2 definitions to osversion package

This change adds three new definitions to the osversion package. All
three definitions are all of the 21H2 builds across Windows 10, Windows Server
and Windows 11, which all have different build numbers.

The approach taken was to add a suffix to the definitions with Win10, Server and
Win11 respectively.

Signed-off-by: Daniel Canter <[email protected]>
(cherry picked from commit 3a8cd1e)
Signed-off-by: Daniel Canter <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants