Skip to content

docs: add test-container-proxy to make help output #2249

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.

Sign up for GitHub

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

Merged
lpcox merged 1 commit into from
Mar 21, 2026
Merged

docs: add test-container-proxy to make help output #2249

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 1 addition & 0 deletions Makefile
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -301,6 +301,7 @@ help:
@echo " test-all - Run all tests (unit + integration)"
@echo " test-serena - Run Serena MCP Server tests (direct connection)"
@echo " test-serena-gateway - Run Serena MCP Server tests (via MCP Gateway)"
@echo " test-container-proxy - Run container proxy integration tests (requires Docker)"

Copilot AI Mar 21, 2026

Copy link

Choose a reason for hiding this comment

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

The help output advertises a test-container-proxy target, but there is no corresponding make target defined anywhere in this Makefile (the only occurrence is this help line). This will mislead users; either add the test-container-proxy: target (and any required .PHONY entry) or change the help text to match the actual existing target name.

Suggested change
@echo " test-container-proxy - Run container proxy integration tests (requires Docker)"

Copilot uses AI. Check for mistakes.
@echo " coverage - Run unit tests with coverage report"
@echo " test-ci - Run unit tests with coverage and JSON output for CI"
@echo " format - Format Go code using gofmt"
Expand Down
Loading