Skip to content

Conversation

@yannisgu
Copy link
Contributor

When an AssemblyInfo.cs contained a version like 1.0.0.* it wasn't recognized by GitVersion. Only 1.0.0.0 or 1.0.0* worked.

@SimonCropp
Copy link
Contributor

can this be tested?

@JakeGinnivan
Copy link
Contributor

There is now a test fixture in master covering this. If you rebase it should be easy to add a test

@yannisgu
Copy link
Contributor Author

Ok I'll add tests

@yannisgu yannisgu force-pushed the fixAssemblyPatching branch from aacc9b6 to 3117804 Compare November 21, 2014 14:17
When an AssemblyInfo contained a version like 1.0.0.* it wasn't recognized
by GitVersion. Only 1.0.0.0 or 1.0.0* worked.
@yannisgu yannisgu force-pushed the fixAssemblyPatching branch from 3117804 to 34385e8 Compare November 21, 2014 14:37
@yannisgu
Copy link
Contributor Author

Tests added, with correct style, now.

JakeGinnivan added a commit that referenced this pull request Nov 25, 2014
Fixed assemblyinfo patching with stars
@JakeGinnivan JakeGinnivan merged commit 74a0530 into GitTools:master Nov 25, 2014
@gep13 gep13 added this to the 2.0.0 milestone Dec 9, 2014
@SimonCropp SimonCropp added the bug label Dec 16, 2014
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