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 AssemblyVersion attribute #629

Conversation

64J0
Copy link
Member

@64J0 64J0 commented Nov 17, 2024

Description

With this PR, I'm adding the AssemblyVersion attribute to Giraffe.fsproj, and updating the DEVGUIDE document, adding a new step to update this attribute whenever a new release is intended.

Related issues

@64J0 64J0 self-assigned this Nov 17, 2024
@64J0
Copy link
Member Author

64J0 commented Nov 17, 2024

@baronfel and @DalekBaldwin, if you have some free time, please take a look at this PR. I think this must be enough to close the issue #521, and later we can revisit this process, to automate it using https://github.com/ionide/KeepAChangelog.

@64J0 64J0 force-pushed the 521-assembly-version-numbers-have-reverted-to-default-1000-ever-since-nuget-version-500-alpha-002 branch from 375bb27 to 94e310d Compare November 17, 2024 19:08
@@ -2,6 +2,7 @@
<PropertyGroup>
<!-- General -->
<AssemblyName>Giraffe</AssemblyName>
<AssemblyVersion>7.0.2</AssemblyVersion>
Copy link
Member Author

Choose a reason for hiding this comment

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

Another potential fix would be to define this property at the dotnet pack command on CI here https://github.com/giraffe-fsharp/Giraffe/blob/master/.github/workflows/publish.yml#L69

@64J0 64J0 merged commit 8b7a39f into master Nov 22, 2024
10 checks passed
@64J0 64J0 deleted the 521-assembly-version-numbers-have-reverted-to-default-1000-ever-since-nuget-version-500-alpha-002 branch November 22, 2024 21:46
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.

Assembly version numbers have reverted to default 1.0.0.0 ever since NuGet version 5.0.0-alpha-002
3 participants