- Notifications
You must be signed in to change notification settings - Fork897
fix issue with log-file arg#13
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.
By clicking “Sign up for GitHub”, you agree to ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Uh oh!
There was an error while loading.Please reload this page.
Conversation
There was a problem hiding this comment.
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
09366fa intomainUh oh!
There was an error while loading.Please reload this page.
fix issue with log-file arg
Context
Small fix to sure that the
--log-filecli argument is honoured.