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 more Makefile targets to help. Add test section #717

Merged
merged 1 commit into from
Oct 1, 2022

Conversation

oscr
Copy link
Contributor

@oscr oscr commented Sep 26, 2022

What this PR does / why we need it:

Adds more Makefile test targets to the make help target and adds a test section.

make help changes to build section and new test section with the pr:

Build
  build            Build manager binary.
  run              Run a controller from your host.
  docker-build     Build docker image with the manager.
  docker-push      Push docker image with the manager.
  cluster-templates-v1beta1  Generate cluster templates for v1beta1

Test
  test             Run unit tests
  test-coverage    Run test-coverage
  agent-test       Run agent tests
  controller-test  Run controller tests
  webhook-test     Run webhook tests
  test-e2e         Run the end-to-end tests

Build section currently:

Build
  build            Build manager binary.
  run              Run a controller from your host.
  docker-build     Build docker image with the manager.
  docker-push      Push docker image with the manager.
  test-e2e         Run the end-to-end tests
  cluster-templates-v1beta1  Generate cluster templates for v1beta1

Copy link
Contributor

@anusha94 anusha94 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@anusha94 anusha94 requested a review from dharmjit September 26, 2022 10:26
Copy link
Contributor

@dharmjit dharmjit left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks! lgtm

@dharmjit dharmjit merged commit 6f957dd into vmware-tanzu:main Oct 1, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants