Skip to content

Respect log level from rc files - #2156

Open
saidM wants to merge 1 commit into
jordansissel:mainfrom
saidM:fix/respect-log-level-from-config
Open

Respect log level from rc files#2156
saidM wants to merge 1 commit into
jordansissel:mainfrom
saidM:fix/respect-log-level-from-config

Conversation

@saidM

@saidM saidM commented Aug 27, 2026

Copy link
Copy Markdown

Fixes #2132

Log level flags defined in .fpm config files were being ignored because option parsing set the default logger level after reading the config options.

This fix ensures log levels specified in configuration files are respected, while still allowing explicit CLI flags to take precedence.

Changes

  • Apply log level settings loaded from config files.
  • Keep CLI arguments as the highest precedence.

Testing

  • Tested passing log level options via .fpm file.
  • Verified CLI flags still override file config as expected.

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.

fpm ignoring --log level

1 participant