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
e2e tests for multiple versions of kube #71
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.
Couple little things inline.
Also, since it is only one test right now, it would be really easy to move to Venom for these. Not a requirement of this PR or a blocker at all, just wanted to point that out.
Codecov Report
@@ Coverage Diff @@
## master #71 +/- ##
=======================================
Coverage 39.19% 39.19%
=======================================
Files 4 4
Lines 199 199
=======================================
Hits 78 78
Misses 112 112
Partials 9 9 Continue to review full report at Codecov.
|
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.
Nit inline. Otherwise looks great. Thanks!
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.
I saw the passing tests in circle. 🎉
I realize now that by squashing down to a single commit I lost @kimschles contributions to this branch. Sorry about that and thanks for helping with this @kimschles !