Skip to content
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 make help command #34

Merged
merged 1 commit into from Jul 9, 2021
Merged

Add make help command #34

merged 1 commit into from Jul 9, 2021

Conversation

msherif1234
Copy link
Contributor

@msherif1234 msherif1234 commented Jun 23, 2021

Signed-off-by: Mohamed Mahmoud mmahmoud@redhat.com

make help
test                            Run tests
test-e2e                        Run e2e tests
manager                         Build manager binary
run                             Run against the configured cluster
install                         Install CRDs into a cluster
uninstall                       Uninstall CRDs from a cluster
deploy                          Deploy controller in the configured cluster
manifests                       Generate manifests e.g. CRD, RBAC etc.
fmt                             Run go fmt against code
vet                             Run go vet against code
generate                        Generate code
docker-build                    Build the docker image
docker-push                     Push the docker image
controller-gen                  Find or download controller-gen
generate-metallb-manifests      Generate metallb manifests
validate-metallb-manifests      Validate metallb manifests
help                            Show this help

If you are adding new make target just add a comment starts with ## beside the new target and describe what this command do, then confirm you can see the output with make help

Makefile Show resolved Hide resolved
Makefile Show resolved Hide resolved
Signed-off-by: Mohamed Mahmoud <mmahmoud@redhat.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants