machine-controller-manager icon indicating copy to clipboard operation
machine-controller-manager copied to clipboard

Introduce `verify` target

Open aaronfern opened this issue 1 year ago • 1 comments

How to categorize this issue?

/area ops-productivity /kind enhancement /priority 3

What would you like to be added: Currently we have three separate make targets that we want to run before we merge a PR or say that a PR is ready to be consumed

  • make check
  • make build
  • make test

Since we need all three to run and pass, we can combine them into a single target make verify. This target should run all the above mentioned make targets

Why is this needed: This is needed to consolidate multiple make targets. This will also get us to simplify our pipeline definitions as it gives us one step to configure instead of 3 and hence only one golang version needs to be specified and this can be moved to branch.cfg


Changes/PRs to be filed to close this story

  • [ ] MCM
  • [ ] MCM-provider-extensions
    • [ ] AWS
    • [ ] Azure
    • [ ] GCP
    • [x] Openstack - Already done
    • [ ] Alicloud

aaronfern avatar Jul 12 '24 06:07 aaronfern

@aaronfern Label area/todo does not exist.

gardener-robot avatar Jul 12 '24 06:07 gardener-robot