Skip to content
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

Add 21H2 definitions to osversion package #1205

Merged
merged 2 commits into from
Nov 11, 2021

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 dcanter@microsoft.com

@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 <dcanter@microsoft.com>
@dcantah dcantah force-pushed the osversion-21h2 branch 2 times, most recently from eb41aaa to 5b0de38 Compare November 10, 2021 22:13
Signed-off-by: Daniel Canter <dcanter@microsoft.com>
@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 <dcanter@microsoft.com>
(cherry picked from commit 3a8cd1e)
Signed-off-by: Daniel Canter <dcanter@microsoft.com>
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