Skip to content

Apply Version Number to Assembly #31

Closed
rpriest-besys wants to merge 16 commits intoChangemakerStudios:developfrom
rpriest-besys:update_asm_ver
Closed

Apply Version Number to Assembly #31
rpriest-besys wants to merge 16 commits intoChangemakerStudios:developfrom
rpriest-besys:update_asm_ver

Conversation

@rpriest-besys
Copy link
Copy Markdown

Hello,

I would like to submit this PR to address Issue #28. As described in the issue, it appears that the assembly versioning was dropped in the most recent builds. As such the libraries are defaulting to 1.0.0.0 for File Version and Assembly Version.

Please take a look at these changes and accept this PR. I must admit, I was not sure how to properly test this change. If there is a way to produce a test build, on Github for a PR, please send me the instructions and I will do so.

Otherwise, if you could advise what the proper changes should be, that would be great!

Cheers!

@rpriest-besys rpriest-besys marked this pull request as draft March 20, 2025 22:11
@rpriest-besys
Copy link
Copy Markdown
Author

rpriest-besys commented Mar 21, 2025

Ok - I figured out how to correct the GitVersion.yml file, so It builds with it generating the version number information. I had to change the mode specification from 'Mainline' to 'ContinousDelivery'. 'Mainline' is not an accepted value for that. And in the pack command, I had to correct where to look up version number. So that all works, and I have a successful build.

Also, I had to merge my branch target_fx_update, into this one. So, this PR should satisfy both

I had to do that, because I had already fixed the update the target fx versions.

I was not able to verify the version numbers in the Common.Logging.Serilog.dll file, because this workflow does not seem to have any artifacts to download ? So, if you could check it or instruct me as to how I can check it, that would be great.

Thank you!

@rpriest-besys rpriest-besys marked this pull request as ready for review March 21, 2025 18:35
@Jaben Jaben closed this Mar 29, 2025
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