-
Notifications
You must be signed in to change notification settings - Fork 28
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
document build and developer process. #188
Comments
@travisn , @subhamkrai - what do you think? - is this something that should be documented? |
I can confirm @BlaineEXE's suggestion from the huddle: directly invoking |
Hi, could you add this as part of the developer guide? Developer changesWe are adding unittests in order to ensure that the destroy-cluster command is working as we expected, $ make generate
generating mocks...
completed
$ Run testTo run the tests you should to execute $ make test
running unit tests
go test ./...
? github.com/rook/kubectl-rook-ceph/cmd [no test files]
ok github.com/rook/kubectl-rook-ceph/cmd/commands (cached)
ok github.com/rook/kubectl-rook-ceph/pkg/crds (cached)
? github.com/rook/kubectl-rook-ceph/pkg/debug [no test files]
? github.com/rook/kubectl-rook-ceph/pkg/dr [no test files]
? github.com/rook/kubectl-rook-ceph/pkg/exec [no test files]
? github.com/rook/kubectl-rook-ceph/pkg/health [no test files]
? github.com/rook/kubectl-rook-ceph/pkg/k8sutil [no test files]
? github.com/rook/kubectl-rook-ceph/pkg/logging [no test files]
? github.com/rook/kubectl-rook-ceph/pkg/rook [no test files]
ok github.com/rook/kubectl-rook-ceph/pkg/mons (cached)
$ |
The docs of this project doo not feature a guide how to build and test the plugin locally,
including instructions how to use a locally built plugin with kubectl.
The text was updated successfully, but these errors were encountered: