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

Test project not using $(PackageVersion_YamlDotNet) #2

Merged

Conversation

Farthom
Copy link

@Farthom Farthom commented May 9, 2018

Looks like I missed this error in my last contribution. The GitVersionCore.Tests project seems to not use the common $(PackageVersion_YamlDotNet) I assume this was unintentional? This is generating a package version mismatch now as a result.
This PR just changes 4.2.1 -> $(PackageVersion_YamlDotNet)

Sorry for missing this before, I should have done better diligence. I hope this build fares better.

@dazinator
Copy link
Owner

No problem. Unfortunately the build server doesn't seem to build these PR's so I can't check if the PR has broken the build before merging. I can build it locally and thats about it. Change looks sensible however, so I will merge.

@dazinator dazinator merged commit a376842 into dazinator:feature/netstandard May 9, 2018
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