Skip to content

Comments

Correct go run commands to run the logging file#566

Merged
findleyr merged 1 commit intomodelcontextprotocol:mainfrom
AdrielVelazquez:patch-1
Oct 11, 2025
Merged

Correct go run commands to run the logging file#566
findleyr merged 1 commit intomodelcontextprotocol:mainfrom
AdrielVelazquez:patch-1

Conversation

@AdrielVelazquez
Copy link
Contributor

@AdrielVelazquez AdrielVelazquez commented Oct 9, 2025

Readme update and clarification

Targeting only the main.go doesn't add the logging file and causes errors when finding additional functions.

mcp-server-testing main*​ ☸ prod-3-d 33s 
❯ go run main.go server
# command-line-arguments
./main.go:127:24: undefined: LoggingHandler

mcp-server-testing main*​ ☸ prod-3-d 
❯ go run . server      
2025/10/09 11:33:13 MCP server listening on localhost:8000
2025/10/09 11:33:13 Available tool: cityTime (cities: nyc, sf, boston)

Instead of running main.go run all files so that they are included in the build.
@findleyr findleyr merged commit 7a89c31 into modelcontextprotocol:main Oct 11, 2025
5 checks passed
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.

2 participants