search

gardener / machine-controller-manager

Declarative way of managing machines for Kubernetes cluster
Apache License 2.0
247 stars 115 forks source link

Introduce `verify` target #930

Open aaronfern opened 1 month ago

aaronfern commented 1 month ago

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

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

gardener-robot commented 1 month ago

@aaronfern Label area/todo does not exist.