-
Notifications
You must be signed in to change notification settings - Fork 15
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 GitHub actions #12
Conversation
4b76859
to
d2bc105
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.
Could we make golangci-lint optionional (or skipped) in this PR and then address the issues in a follow-up.
Some changes required regardless.
d2bc105
to
1de4eb7
Compare
@tariq1890 all your recommendations sound good, but the idea is to have all repos with a similar structure. |
45ce6f5
to
c642cad
Compare
Hey @tariq1890 can we move forward with this PR? |
@elezar PTAL |
@cdesiniotis PTAL |
@ArangoGutierrez this needs a rebase |
Signed-off-by: Carlos Eduardo Arango Gutierrez <[email protected]>
Signed-off-by: Carlos Eduardo Arango Gutierrez <[email protected]>
c642cad
to
1342444
Compare
done |
No description provided.