Skip to content

Conversation

@JaynieBai
Copy link
Member

Fixes #9923

Context

#9789 added a template for custom analyzer. Microsoft.Build version specified in the template should be synced with Version.props during packing. See https://github.com/dotnet/msbuild/pull/9789/files#r1521218723 for more details.
This needs to be done after #9915.

Changes Made

Create a workflow. When there are updates in the file Version.props, the flow will be trigger and check the version. If the version in the template is different form the VersionPrefix in the Version.props. It will create a PR to sync the version between them.

Testing

Tested in the forked repo https://github.com/JaynieBai/msbuild/actions/runs/9852188308

Notes

@JaynieBai JaynieBai requested a review from a team as a code owner July 9, 2024 06:48
@JaynieBai JaynieBai changed the title Jennybai/sync version flow Sync Microsoft.Build version in analyzer template with Version.props in the workflow Jul 9, 2024
Copy link
Member

@JanKrivanek JanKrivanek left a comment

Choose a reason for hiding this comment

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

Overall looks good. I added couple comments for consideration

Copy link
Member

@JanKrivanek JanKrivanek left a comment

Choose a reason for hiding this comment

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

The change is good. We just need to move from the used GH Action. I'll research our options and get back

@JaynieBai JaynieBai requested a review from JanKrivanek August 6, 2024 04:55
Copy link
Member

@JanKrivanek JanKrivanek 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
Thank you!

@JaynieBai JaynieBai merged commit 705a282 into dotnet:main Aug 6, 2024
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.

Sync Microsoft.Build version in analyzer template with Version.props during packing

4 participants