Skip to content

Conversation

@MarcoRossignoli
Copy link
Contributor

@MarcoRossignoli MarcoRossignoli commented Jun 29, 2022

Bump to 17.4.0

completes #3818

Copy link
Member

@Evangelink Evangelink left a comment

Choose a reason for hiding this comment

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

@Haplois Can we share the version somehow between the 2 parts? Or is it possible to create a script that validates the versions produced so we avoid doing a partial bump as we did?

@MarcoRossignoli MarcoRossignoli enabled auto-merge (squash) June 30, 2022 09:26
and because during the test `dotnet test` will run and re-build some of the test projects and at that time the version
from a build parameter would not be available, so I am writing this version from the build.ps1 script to keep it in sync -->
<NETTestSdkVersion>17.4.0-dev</NETTestSdkVersion>
<NETTestSdkVersion>17.3.0-dev</NETTestSdkVersion>
Copy link
Member

Choose a reason for hiding this comment

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

Might be worth to add a comment why this can't be latest version.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Isn't clear from the comment :) ?

@MarcoRossignoli MarcoRossignoli merged commit f5c01b8 into microsoft:main Jun 30, 2022
@MarcoRossignoli MarcoRossignoli deleted the bumpver branch June 30, 2022 11:54
@Haplois
Copy link
Contributor

Haplois commented Jun 30, 2022

@Haplois Can we share the version somehow between the 2 parts? Or is it possible to create a script that validates the versions produced so we avoid doing a partial bump as we did?

I will update our build script to build and update the version accordingly like we do in TestFx when we provide external versions.

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.

3 participants