-
Notifications
You must be signed in to change notification settings - Fork 90
Conversation
Welcome @uablrek! |
And IMO the "version" command and variable belongs in |
thanks :) /lgtm |
Signed-off-by: Lars Ekman <lars.g.ekman@est.tech>
@mcluseau PTAL. I had to amend the commit message. |
/lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: mcluseau, uablrek The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
Adds;
The
main.version
string is defined at compile time using-X main.version=(something)
. Example;Now the git commit is printed;
When developing when the commit is not updated a
date
may be more useful;NOTE; I have deliberately made the printout clean. No "Kpng version:" prefix or something. The version will most likely be interpreted by some script or test-report tool and unnecessary decorations are just annoying. And humans understand it just as good.