Skip to content

fix issue with log-file arg - #13

Merged
juruen merged 2 commits into
mainfrom
juruen/fix-log-file
Mar 14, 2025
Merged

fix issue with log-file arg#13
juruen merged 2 commits into
mainfrom
juruen/fix-log-file

Conversation

@juruen

Copy link
Copy Markdown
Collaborator

Context

Small fix to sure that the --log-file cli argument is honoured.

CopilotAI review requested due to automatic review settings March 14, 2025 18:52

CopilotAI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull Request Overview

This PR fixes the issue with the log-file CLI argument by ensuring that the flag is properly bound to viper.

  • Removes the persistent pre-run binding from the server command.
  • Adds the flag binding in the global initialization of persistent flags, ensuring it applies to all commands.
Files not reviewed (1)
  • go.mod: Language not supported

Tip: Copilot only keeps its highest confidence comments to reduce noise and keep you focused. Learn more

@juruen
juruen merged commit 09366fa into mainMar 14, 2025

@SoCal909SoCal909 left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Verify

DaleSeo pushed a commit to DaleSeo/github-mcp-server that referenced this pull request Oct 24, 2025
Signed-off-by: Ivan Pedrazas <ivan.pedrazas@docker.com>
imfromfuture3000-Android referenced this pull request in imfromfuture3000-Android/github-mcp-server Mar 1, 2026
…ct-spawner-kxmx14
Add CryptoGene agent blueprint, contract scanner, and deployment workflow hardening
Sign up for freeto 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.

4 participants

@juruen@SamMorrowDrums@SoCal909