-
Notifications
You must be signed in to change notification settings - Fork 0
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
Initial CI workflow #12
Conversation
e1c94c7
to
5cc9dfb
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.
LGTM
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.
Looks good one minor optional change there's a newer version of actions/setup-go
available
95491ba
to
fa64517
Compare
Don't forget to add the new lines at the end of files, also should we also add here the gorelease? |
Added a newline at the end! I suggest creating a separate PR for the build/release (after discussing with the team first). |
Part of this #10