Skip to content

Updating release history related to PR #607#609

Merged
michaelcfanning merged 7 commits into
mainfrom
updating-release-history
Jul 8, 2022
Merged

Updating release history related to PR #607#609
michaelcfanning merged 7 commits into
mainfrom
updating-release-history

Conversation

@eddynaka

Copy link
Copy Markdown
Collaborator

Changes

This change will add an entry to the latest PR that got merged (#607).

When we release it, we will use the latest available in the windows-2022 VM:
https://github.com/actions/virtual-environments/blob/main/images/win/Windows2022-Readme.md#installed-windows-sdks

@michaelcfanning michaelcfanning left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

🕐

Comment thread Src/ReleaseHistory.md Outdated

## Unreleased

- RE2.Native will compile with the latest Windows SDK 10.0.* installed. [#607](https://github.com/microsoft/sarif-pattern-matcher/pull/607)

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Native

We don't want this note. Customers should not receive an arbitrary Windows SDK, and we don't need to document a change that's relevant only to dev builds.

@michaelcfanning

Copy link
Copy Markdown
Member

Hello! What's happened here? We were going to pin our pipeline builds to a specific Win SDK and then document that version in our release notes. Is that still happening? :)

Update release note to provide placeholder for actual windows sdk version used in release builds.
Comment thread Src/ReleaseHistory.md

## Unreleased

- RE2.Native will now compile in all environments with the latest Windows SDK 10.0.* installed. [#607](https://github.com/microsoft/sarif-pattern-matcher/pull/607). Our current release pipelines build NuGet packages with Windows SDK version 10.0.22000.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

10.0.22000

@eddynaka, we need to update this version w/what's actually used in our release/deployment pipeline build agents.

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

The version that you mentioned is the version that we will be using.
So, the text is accurate.

Thanks for updating it.

@michaelcfanning michaelcfanning enabled auto-merge (squash) June 6, 2022 20:26
@michaelcfanning michaelcfanning disabled auto-merge July 8, 2022 15:54
@michaelcfanning michaelcfanning merged commit 3885b86 into main Jul 8, 2022
@eddynaka eddynaka deleted the updating-release-history branch June 26, 2023 21:40
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.

2 participants