-
Notifications
You must be signed in to change notification settings - Fork 72
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
Printing version #612
Printing version #612
Conversation
"1.0.0+2cfaef9576731e7852852518b730349341b069d9"
Hey @Thorium thanks for your PR! Looks good in principle, just gonna point out some nits. |
Done. |
So I assume you tested your own PR? Just making sure, because it's very surprising to me that you can get the commit hash out of some reflection API. |
I originally took the code from Paket repository, and that is used a lot :-) |
This codebase is using Paket, I think that puts commit hash to AssemblyInfo style attributes. |
The binaries are sourcelinked, and the sourcelink tools can insert the git commit info into the assembly version information. |
It uses Paket to restore, I doubt it uses Paket to pack, because it is Fake the tool that is packing, right? See https://github.com/fsprojects/FSharpLint/blob/master/build.fsx#L124
Oh interesting! I guess I missed the commit that introduced SourceLink support, must be old (before I became maintainer); will investigate. |
This will add --version argument for printing version, with GitHub commit-id e.g.
"1.0.0+2cfaef9576731e7852852518b730349341b069d9"
Fix for #611