-
Notifications
You must be signed in to change notification settings - Fork 103
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 version flag and build amd64 and arm64 targets #86
Conversation
gibing some love to this project and also preparing to the ownership migration to cloud sre team, that will run push proxy in a container, so preparing all the autobahn for future usage |
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.
other than the PHONY comment it lgtm
61cbd41
to
e412903
Compare
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.
minor few comments
|
||
package-linux-amd64: |
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.
better to make this a script and just change the flags accordingly so it can be repeatable and avoid copy/paste in two places in case we change something?
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.
agree, maybe we can do in a follow up
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.
yeah let's plan for this
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, let's fix the script too in a follow up PR
Summary
version demo:
$ ./bin/mattermost-push-proxy --version GitVersion: v5.22.5-7-g35e4ca9 GitCommit: 35e4ca962a1a44b151387e27455b3bf013b711fe GitTreeState: clean BuildDate: '2021-10-19T12:33:02Z' GoVersion: go1.17.2 Compiler: gc Platform: darwin/amd64
Ticket Link
JIRA https://mattermost.atlassian.net/browse/DOPS-22