Skip to content

Conversation

@dcantah
Copy link
Contributor

@dcantah dcantah commented Oct 6, 2021

This reverts commit 7b6e3da that added
a windows server 2022 build number. The reason
for this is because the tag 21h2 actually refers to a couple different
Windows builds now unfortunately. It's the tag for the latest Windows 10
update, for Windows 11, and Windows Server 2022. We're looking into how
best to actually name these going forward if we're going to account for an
event like this again.

@dcantah dcantah requested a review from a team as a code owner October 6, 2021 20:01
This reverts commit 7b6e3da. The reason
for this is because the tag 21h2 actually refers to a couple different
Windows builds now unfortunately. It's the tag for the latest Windows 10
update, for Windows 11, and Windows Server 2022. We're looking into how
best to actually name these going forward if we're going to account for an
event like this again.

Signed-off-by: Daniel Canter <[email protected]>
Copy link

@katiewasnothere katiewasnothere left a comment

Choose a reason for hiding this comment

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

Jeez, that sounds like a mess. This LGTM.

@dcantah
Copy link
Contributor Author

dcantah commented Oct 6, 2021

fyi @adelina-t, sorry about this :(. We became aware of it after we'd checked it in. I'll ping you once we come up with something here

@dcantah dcantah merged commit 2a4f814 into microsoft:master Oct 6, 2021
@thaJeztah
Copy link
Contributor

would it be an idea to have multiple consts with the same value? (fwiw, older consts also could refer to multiple things (windows home, server, etc), so perhaps just a docs comment would "work"

@dcantah
Copy link
Contributor Author

dcantah commented Oct 7, 2021

@thaJeztah The windows 10, windows 11 and windows server 21h2 all have different build numbers 😐 The idea we'd floated around so far was to have some sort of prefix to denote each. W11, W10 and WS or something similar.

@thaJeztah
Copy link
Contributor

Ah! I misunderstood, and thought they were the same build number, but different versions, but it's the reverse. Gotcha

@dcantah
Copy link
Contributor Author

dcantah commented Oct 7, 2021

@thaJeztah Yea, it's the worst case scenario 😔

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