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] Typo in GitVersion.MsBuild.targets #2582

Closed
MarkusHorstmann opened this issue Feb 5, 2021 · 5 comments · Fixed by #2583
Closed

[Bug] Typo in GitVersion.MsBuild.targets #2582

MarkusHorstmann opened this issue Feb 5, 2021 · 5 comments · Fixed by #2583
Labels
Milestone

Comments

@MarkusHorstmann
Copy link
Contributor

MarkusHorstmann commented Feb 5, 2021

<DefineConstants Condition=" '$(GitVersion_PreReleaseLabelWithDash)' != '' ">GitVersion_PreReleaseLabelWithDash=$(GitVersion_PreReleaseLabeWithDashl);$(DefineConstants)</DefineConstants>
should likely be
<DefineConstants Condition=" '$(GitVersion_PreReleaseLabelWithDash)' != '' ">GitVersion_PreReleaseLabelWithDash=$(GitVersion_PreReleaseLabelWithDash);$(DefineConstants)</DefineConstants>

Noticed doing other changes, but don't have a way to test this.

Expected Behavior

Unknown

Actual Behavior

Unknown

Possible Fix

See above.

Steps to Reproduce

Unknown

Context

Code inspection

Your Environment

@asbjornu
Copy link
Member

asbjornu commented Feb 5, 2021

GitVersion_PreReleaseLabeWithDashl does not look right indeed. Could you please submit a PR that fixes this? 🙏🏼

@MarkusHorstmann
Copy link
Contributor Author

Happy to include it in my current PR, or do you prefer separate PRs for different fixes?

@asbjornu
Copy link
Member

asbjornu commented Feb 5, 2021

Since #2581 does not deal with the MSBuild task directly, fixing the typo in a separate PR would be preferable. 🙏🏼

@MarkusHorstmann
Copy link
Contributor Author

Ok, I'll revert it there and kick off a separate PR.

MarkusHorstmann added a commit to MarkusHorstmann/GitVersion that referenced this issue Feb 5, 2021
@asbjornu asbjornu added this to the 5.6.5 milestone Feb 5, 2021
MarkusHorstmann added a commit to MarkusHorstmann/GitVersion that referenced this issue Feb 5, 2021
…ion_PreReleaseLabelWithDash"

This reverts commit f2b6131.
asbjornu added a commit that referenced this issue Feb 6, 2021
#2582: GitVersionMsBuild.Targets: fix typo in GitVersion_PreReleaseLabelWithDash
arturcic pushed a commit that referenced this issue Feb 7, 2021
Revert "#2582: GitVersionMsBuild.Targets: fix typo in GitVersion_PreReleaseLabelWithDash"

This reverts commit f2b6131.
arturcic pushed a commit to MarkusHorstmann/GitVersion that referenced this issue Feb 7, 2021
…ion_PreReleaseLabelWithDash"

This reverts commit f2b6131.
arturcic pushed a commit that referenced this issue Feb 7, 2021
@github-actions
Copy link

github-actions bot commented Feb 7, 2021

🎉 This issue has been resolved in version 5.6.5 🎉
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
2 participants