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

[Bug] Inconsistency between GitVersion.MsBuild and GitVersion.Tool #2813

Closed
vhatsura opened this issue Aug 16, 2021 · 4 comments · Fixed by #2817
Closed

[Bug] Inconsistency between GitVersion.MsBuild and GitVersion.Tool #2813

vhatsura opened this issue Aug 16, 2021 · 4 comments · Fixed by #2817
Labels
Milestone

Comments

@vhatsura
Copy link
Contributor

Describe the bug
GitVersion.Tool sets NuGetVersionV2 as Version attribute in project file, however, GitVersion.MsBuild uses FullSemVer.

Expected Behavior

GitVersion.MsBuild should use NuGetVersionV2 as Version attribute in project file.

Actual Behavior

FullSemVer is used as the value for the Version attribute in the project file.

Possible Fix

Change a value in targets to $(GitVersion_NuGetVersionV2)

Steps to Reproduce

Context

It prevents from setting proper version for NuGet package with padded prerelease tag. As a workaround, it's possible to disable UseFullSemVerForNuGet.

Your Environment

  • Version Used: 5.6.11
@vhatsura vhatsura added the bug label Aug 16, 2021
@asbjornu
Copy link
Member

I agree it should be consistent, but I'm not sure whether GitVersion.Tool should start using FullSemVer or GitVersion.MsBuild should start using NuGetVersionV2. But one of them should probably change to the other. Thoughts, @arturcic?

@arturcic
Copy link
Member

I agree that we need to make it consistent, but I would go for $(GitVersion_NuGetVersion) for both cases.
@vhatsura are you in a position to make this change and submit a PR?

@vhatsura
Copy link
Contributor Author

vhatsura commented Aug 17, 2021

I agree that we need to make it consistent, but I would go for $(GitVersion_NuGetVersion) for both cases.

@vhatsura are you in a position to make this change and submit a PR?

Yeap, I’m open to implement such changes.

@asbjornu asbjornu added this to the 5.x milestone Sep 23, 2021
@arturcic arturcic modified the milestones: 5.x, 5.8.0 Oct 3, 2021
@github-actions
Copy link

🎉 This issue has been resolved in version 5.8.0 🎉
The release is available on:

Your GitReleaseManager bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants