Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Add
--log-levelso commands such asopenserp search bing "golang" --log-level errorcan suppress warnings, including the warning emitted when no default config file exists.The setting also supports
app.log_levelandOPENSERP_APP_LOG_LEVEL, with CLI > environment > config precedence. A non-empty value overrides the logging threshold selected by quiet/verbose/debug; an empty value retains existing defaults. Existing logger output destinations and browser settings are preserved.Closes #44.
Why
The missing-config warning was logged before normal logger initialization. The merged log level is now applied before startup warnings and reapplied after logger setup. Invalid levels return a configuration error before search starts.
Testing
make fmtmake test(full suite with race detector)make lint(go vet and golangci-lint v1.64.8)go buildnpx --yes @redocly/cli lint docs/openapi.yaml(passes with 9 warnings in the unchanged specification)Live engine integration tests were not run; the change is confined to CLI logging configuration.
Checklist