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

Update Bug Report template #6673

Merged
merged 3 commits into from
Feb 28, 2022
Merged

Update Bug Report template #6673

merged 3 commits into from
Feb 28, 2022

Conversation

LeftTwixWand
Copy link
Contributor

Description

  • Structured OS Version and Build in the Bug Report Template
  • Added missing latest OS versions:
    Windows 11: Build 22000
    Windows 10: Build 19044

Motivation and Context

First - absence of the latest OS build is pretty confusing.
Also, I think, that restructuring of the OS version chapter will improve contributor's readability.

Screenshot:

image

Added Windows 19044 build to the Bug report template
Improved OS version and build readability
@ghost ghost added the needs-triage Issue needs to be triaged by the area owners label Feb 3, 2022
@StephenLPeters
Copy link
Contributor

@gabbybilka thoughts?

@StephenLPeters StephenLPeters added documentation An issue with existing documentation or a request for documenation of a new topic and removed needs-triage Issue needs to be triaged by the area owners labels Feb 10, 2022
@StephenLPeters
Copy link
Contributor

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

- "Fall Creators Update (16299)"
- "Creators Update (15063)"
- "Windows Insider Build (xxxxx)"
- "Windows 11: Build 22000"
Copy link
Member

Choose a reason for hiding this comment

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

I think it would be helpful to use the code names as well for the Windows 10 builds as they are relatively well known: https://en.wikipedia.org/wiki/Windows_10_version_history#Channels

I propose this format:
Windows 10: Build 19044 --> Windows 10 21H2: Build 19044

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Hey, sound good, but I have a question. What's better to use - windows Version or Codename?

The screenshots, how it looks like:
image

Copy link
Member

Choose a reason for hiding this comment

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

I like version — good call!
Windows 11 (21H2): Build 22000

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I've updated the template with the last commit

Now all the Windows versions have the same style:
`Windows [version] (version abbreviation): Build xxxxx`

Example:
Windows 11 (21H2): Build 22000
Copy link
Member

@gabbybilka gabbybilka left a comment

Choose a reason for hiding this comment

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

Looks good to me, thanks for contributing! 🙂

@LeftTwixWand
Copy link
Contributor Author

LeftTwixWand commented Feb 16, 2022

Thank you for such cool technology)

@StephenLPeters, I suppose, that it's needed to run Azure pipelines again, to finish the PR?

@azure-pipelines
Copy link

Commenter does not have sufficient privileges for PR 6673 in repo microsoft/microsoft-ui-xaml

@StephenLPeters
Copy link
Contributor

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@LeftTwixWand
Copy link
Contributor Author

How can I sign the CLA?

@ghost ghost added the needs-triage Issue needs to be triaged by the area owners label Feb 28, 2022
@StephenLPeters StephenLPeters merged commit 212f038 into microsoft:main Feb 28, 2022
@LeftTwixWand LeftTwixWand deleted the patch-1 branch February 28, 2022 21:56
@StephenLPeters StephenLPeters removed the needs-triage Issue needs to be triaged by the area owners label Mar 2, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation An issue with existing documentation or a request for documenation of a new topic
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants