-
Notifications
You must be signed in to change notification settings - Fork 5k
Reportcard again #5679
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
Reportcard again #5679
Conversation
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: afbjorklund The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
Codecov Report
@@ Coverage Diff @@
## master #5679 +/- ##
==========================================
- Coverage 36.58% 36.54% -0.05%
==========================================
Files 110 110
Lines 8096 8114 +18
==========================================
+ Hits 2962 2965 +3
- Misses 4745 4760 +15
Partials 389 389
|
Mostly by using the "extract method" pattern. Here is the report (from gocyclo), before: 21 cmd runStart cmd/minikube/cmd/start.go:272:1 18 cmd validateNetwork cmd/minikube/cmd/start.go:982:1 16 cmd runDelete cmd/minikube/cmd/delete.go:90:1 16 cmd deleteProfile cmd/minikube/cmd/delete.go:177:1
570e894
to
d635b8d
Compare
Supposed to be removed in 361222f, along with some other random changes to comments and such.
Improve gofmt (!), golint and gocyclo.
The integration tests are tricky, they are by their nature sequential and require setup/takedown.