-
-
Notifications
You must be signed in to change notification settings - Fork 9
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
feat: Add Github actions CI #8
Conversation
Signed-off-by: Ashish Malik <b218008@iiit-bh.ac.in>
Signed-off-by: Ashish Malik <b218008@iiit-bh.ac.in>
Signed-off-by: Ashish Malik <b218008@iiit-bh.ac.in>
Signed-off-by: Ashish Malik <b218008@iiit-bh.ac.in>
Signed-off-by: Ashish Malik <b218008@iiit-bh.ac.in>
main.go
Outdated
@@ -5,11 +5,12 @@ import ( | |||
"crypto/tls" | |||
"flag" | |||
"fmt" | |||
"github.com/golang/glog" |
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.
sort imports
Signed-off-by: Ashish Malik <b218008@iiit-bh.ac.in>
Made the necessary changes! I used the gofmt to style the code, but still it is showing that it is not yet formatted in the goreportcard- https://goreportcard.com/report/github.com/ashish493/k8s-authz |
@ashish493 goreportcard is too slow at my side. And I don't think it's 100% correct. Let's merge first and see how it goes. |
Yeah, its really very slow. Also, the CI badge is not rendered properly, I will replace it in the next PR. |
🎉 This PR is included in version 1.0.0 🎉 The release is available on GitHub release Your semantic-release bot 📦🚀 |
Fix: #7
@hsluoyz ,
Added the CI for build, lints, tests and for semantic release.
Added goreportcard,coverall and CI build badges for Readme
Fixed the minor bugs.
As of now, build is passing successfully. However, the lints are failing because of some tests related issues. I also have to add an e2e test to check complete working of our application.
I will be doing these tasks in my next PR.