quality-of-life: Improve local development tooling #270
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR updates the
Makefile
with.PHONY
targets, better error handling for deploy/undeploy, and addsREADME.md
sections on cleanup, tests, and linting.Makefile Enhancements:
Added
.PHONY
declarations to multiple targets (local-run
,build-operator
,build
,run
,image-build
,image-push
,deploy
,undeploy
,test-e2e-wait-for-stable-state
,test-e2e-debug-cluster
,lint
) to ensure proper execution and avoid conflicts with files of the same name. [1] [2] [3] [4] [5]Improved the
deploy
target to create thecert-manager-operator
namespace if it does not exist, ensuring smoother deployment.Enhanced the
undeploy
target to ignore not-found errors when deleting resources, improving robustness.Added a check in the
lint
target to ensuregolangci-lint
is installed before running, providing a clearer error message if it is missing.Documentation Updates:
Added a "Cleaning up the deployment" section in
README.md
, explaining how to remove the Cert Manager Operator and its associated resources usingmake undeploy
.Introduced a "Running tests locally" section in
README.md
, detailing how to execute unit tests and generate a coverage report.Added a "Linting the code" section in
README.md
, describing how to runmake lint
to check for linting issues usinggolangci-lint
.