-
Notifications
You must be signed in to change notification settings - Fork 333
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
Bump to 17.4.0 #3831
Bump to 17.4.0 #3831
Conversation
There was a problem hiding this 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?
@@ -11,7 +11,7 @@ | |||
<!-- this version also needs to be "statically" readable because the test fixture will inspect this file for the version | |||
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> |
There was a problem hiding this comment.
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.
There was a problem hiding this comment.
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 :) ?
I will update our build script to build and update the version accordingly like we do in TestFx when we provide external versions. |
Bump to 17.4.0
completes #3818