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

Add 'precision' setting for the NuGet package version #719

Merged
merged 4 commits into from Feb 15, 2022

Conversation

ap0llo
Copy link
Contributor

@ap0llo ap0llo commented Feb 5, 2022

Adds a precision setting to the nugetPackageVersion section in version.json which defaults to build (keeping the existing default behavior).

This allows configuring Nerdbank.GitVersioning to include a 4th version component by setting the precision to revision.

For completeness this also allows to include only the major and/or minor versions in the package version - though I do not know how useful this is.

Fixes #401

@ap0llo
Copy link
Contributor Author

ap0llo commented Feb 6, 2022

Please don't merge this yet, I've had an alternative idea to resolve #401 which I want to explore first

Copy link
Collaborator

@AArnott AArnott left a comment

Choose a reason for hiding this comment

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

This looks very well done. Thank you for all the care you took in it. You mentioned holding off on this, but per our conversation on the github issue are you feeling ready to merge this?
Just a small set of test cases to add as commented below, please.

For the test "CanSetPrecisionForNuGetPackageVersion", add test cases to verify the NuGet package version is computed correctly when a static prerelease tag is configured in version.json
@ap0llo
Copy link
Contributor Author

ap0llo commented Feb 13, 2022

This looks very well done. Thank you for all the care you took in it. You mentioned holding off on this, but per our conversation on the github issue are you feeling ready to merge this?

Yes, I was referring to that conversation. This PR is ready to merge in my opinion.

Just a small set of test cases to add as commented below, please.

I added some additional test cases with commit a6e4aeb

@AArnott AArnott added this to the v3.5 milestone Feb 15, 2022
@AArnott AArnott merged commit d9ed2fe into dotnet:master Feb 15, 2022
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.

Support height in fourth version part for NuGet
2 participants