Skip to content

Validation: Add make verify-fast and make verify - #135

Closed
robbycochran wants to merge 1 commit into
mainfrom
batch5-pr
Closed

Validation: Add make verify-fast and make verify#135
robbycochran wants to merge 1 commit into
mainfrom
batch5-pr

Conversation

@robbycochran

Copy link
Copy Markdown
Collaborator

Summary

Add two new verification targets to the Makefile that do not require Docker, Kind, or OpenShell:

  • make verify-fast: Quick verification with vet + lint (basic code quality checks)
  • make verify: Full verification with vet + lint + test-suite (includes configuration test suite)

Validation

Both targets validated:

  • make verify-fast passes
  • make verify passes with full test-suite output

Files Changed

  • Makefile: Added verify-fast and verify targets, updated .PHONY targets list

Risks

None - purely additive, no breaking changes to existing targets.

Fast verification (vet + lint) without Docker/Kind/OpenShell.
Full verification adds config test-suite.

Validation: make verify-fast and make verify pass.
@coderabbitai

coderabbitai Bot commented Sep 4, 2026

Copy link
Copy Markdown

Important

Draft PR not reviewed

Draft PRs are not automatically reviewed by default.

  • Trigger a manual review

To automatically review draft PRs, update your CodeRabbit configuration:

reviews:
  auto_review:
    drafts: true

Comment @coderabbitai help to get the list of available commands.

@robbycochran

Copy link
Copy Markdown
Collaborator Author

Superseded by #136, which consolidates this change (and the other three batch drafts) into a single PR against main.

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.

1 participant