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

Align github and nuget release versions #1416

Merged

Conversation

daveMueller
Copy link
Collaborator

closes #1122
closes #1413

@daveMueller
Copy link
Collaborator Author

I'm not sure about how the tag for the github release is created? Even though I change the AssemblyVersion in coverlet.core.csproj the assembly itself gets the number specified in version.json.

grafik

I'm also not really able to see how or if coverlets' azure pipeline is setting the release tag? Doesn't seem so. Is the github release created manually? @MarcoRossignoli @petli
Then what is the reason for the AssemblyVersion in coverlet.core.cspro at all? Couldn't we just drop it?

@MarcoRossignoli
Copy link
Collaborator

MarcoRossignoli commented Nov 24, 2022

I'm not sure about how the tag for the github release is created?

I'm creating it manually today using the version inside src/coverlet.core/coverlet.core.csproj

I'm also not really able to see how or if coverlets' azure pipeline is setting the release tag?

we're using this manual workflow to release https://github.com/coverlet-coverage/coverlet/blob/master/Documentation/ReleasePlan.md#how-to-manually-release-packages-to-nugetorg with Nerdbank.GitVersioning https://github.com/dotnet/Nerdbank.GitVersioning

The nightly is generated using @tonerdo pipeline and the setup is here https://github.com/coverlet-coverage/coverlet/blob/master/eng/azure-pipelines-nightly.yml

@daveMueller
Copy link
Collaborator Author

Thanks @MarcoRossignoli. I think I have everything ready now. I already checked the pipelines and it seems that they also don't use the version in src/coverlet.core/coverlet.core.csproj. Thus I think it's safe to just drop it.

@daveMueller daveMueller marked this pull request as ready for review November 26, 2022 00:29
@daveMueller
Copy link
Collaborator Author

/azp run

@azure-pipelines
Copy link

Commenter does not have sufficient privileges for PR 1416 in repo coverlet-coverage/coverlet

@MarcoRossignoli
Copy link
Collaborator

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@MarcoRossignoli MarcoRossignoli merged commit cb420d2 into coverlet-coverage:master Jan 2, 2023
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.

Allign published nuget package version to github release version Sync nuget and github release versions
2 participants