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

Bump version to v2.0 #999

Merged
merged 1 commit into from
Mar 16, 2017
Merged

Bump version to v2.0 #999

merged 1 commit into from
Mar 16, 2017

Conversation

nguerrera
Copy link
Contributor

Aligning SDK version with CLI version, which allows some VS compat goop in Common.props to go away.

@dsplaisted @livarcocc @eerhardt

@@ -28,7 +28,7 @@
<!-- Reset $(BuildEpoch) whenever $(VersionPrefix) increments. We subtract this from YYYYMMDD portion of build
number below to obtain the fourth part of file version that must fit in 16 bits. We can produce builds
for seven years from every epoch reset. -->
<VersionPrefix Condition="'$(VersionPrefix)' == ''">1.1.0</VersionPrefix>
<VersionPrefix Condition="'$(VersionPrefix)' == ''">2.0.0</VersionPrefix>
<BuildEpoch>20170101</BuildEpoch>
Copy link
Member

Choose a reason for hiding this comment

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

Should this get reset, like the above comment indicates?

Copy link
Member

Choose a reason for hiding this comment

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

Or I guess it is the same year, so no real reason to touch it.

Copy link
Contributor

Choose a reason for hiding this comment

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

Maybe fix the comment then?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

It should, but we're so close enough in time to current epoch, it doesn't really matter. (But moving forward, do as I say, not as I do ;))

@nguerrera nguerrera merged commit b5d8d03 into dotnet:master Mar 16, 2017
@nguerrera nguerrera deleted the v2 branch March 16, 2017 22:13
mmitche pushed a commit to mmitche/sdk that referenced this pull request Jun 5, 2020
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.

4 participants