Skip to content

Improved GitVersionTask documentation #792

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

Merged
merged 3 commits into from
Feb 23, 2016
Merged

Improved GitVersionTask documentation #792

merged 3 commits into from
Feb 23, 2016

Conversation

asbjornu
Copy link
Member

Added more information to how GitVersionTask works, a little TL;DR section and rewrote a few section to make the documentation clearer. Addresses #789.

Added more information to how GitVersionTask works, a little TL;DR section and rewrote a few section to make the documentation clearer.

Ensure you remove the `Assembly*Version` attributes from your `Properties\AssemblyInfo.cs` file so as to not get duplicate attribute warnings. Sample default:
At build time a temporary `AssemblyInfo.cs` will be created that contains the appropriate SemVer information.
This will will be included in the build pipeline. Sample default:
Copy link
Member

Choose a reason for hiding this comment

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

@asbjornu you have duplicated will here.

@gep13
Copy link
Member

gep13 commented Feb 22, 2016

@asbjornu looks good, just a couple small comments. If you can fix up, I can merge it in. Thanks!

@asbjornu
Copy link
Member Author

@gep13 Thanks for the review. Everything should be fixed.

@gep13
Copy link
Member

gep13 commented Feb 23, 2016

LGTM! Thanks!

gep13 added a commit that referenced this pull request Feb 23, 2016
@gep13 gep13 merged commit 68adb68 into GitTools:master Feb 23, 2016
@asbjornu asbjornu deleted the feature/msbuild-task-documentation branch February 23, 2016 14:27
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.

2 participants