-
Notifications
You must be signed in to change notification settings - Fork 535
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
Display version, our license, and vendor attributions in version command #407
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, excellent job
Did you check that the binaries created through goreleaser were good?
Adding a don't merge label until we sort out the required Go version. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great job!
@lazartravica You can check the release in my repo |
@Kourin1996 I am unable to open that link - 404 :( |
@lazartravica I guess you can access now |
…command Move version, license, and BSD attributions into params package Remove blank line between license and attributions on license command
I've changed few things:
And you can test sample of binaries/containers in the following release |
Signed-off-by: Luca Georges Francois <luca.georges-francois@epitech.eu>
Signed-off-by: Luca Georges Francois <luca.georges-francois@epitech.eu>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you for not only adding license support, but also for adding an easy way to generate and update them in the future 🙏
I've left a few comments that should be addressed.
I would also love to have a document on our developer docs that details how we use these tools you've provided us with, and when 💯
@zivkovicmilos Please note it's impossible to specify the parent path to |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you for resolving all of these comments 🙏
Looks great now 💯
…e' into feature/print-version-and-license
…rectory to be included in zip
Fixes EDGE-376 and EDGE-391
Pending to merge because this PR requires go v1.16. Please upgrade go version before merging this PR.
Description
This PR sets the proper version which is used in
version
command and addslicense
command to print our license and dependency attributions.version
commandVersion can be embedded by
-ldflags
on build timemake build
): The latest version + current commit short hash will be embeded.license
commandDisplay our license written in
LICENSE
file.license
commandDisplay vendor attributions defined in version/bsd_licenses.json for BSD-2/3 license packages
Result
version
command prints the following results:license
command prints the following results:Changes include
Breaking changes
Please complete this section if any breaking changes have been made, otherwise delete it
Checklist
Additional comments
Please post additional comments in this section if you have them, otherwise delete it