← Back to issue list

fix(log): allows set logging verbosity level from env

View original Github issue

Metadata

Project
craft-application
Number
#289
Type
pull request
State
merged
Author
syu-w
Labels
—
Created
Updated
Closed

Current evaluation

Merged changes that enable log verbosity configuration via the CRAFT_VERBOSITY_LEVEL environment variable, defaulting to BRIEF on invalid input. Approved by two reviewers, passed CI, and updated two files.

Suggested action: —

No scores available.

Issue body

Use `CRAFT_VERBOSITY_LEVEL` to set log emitter level. If the level is invaild, default to BRIEF and continue.

Evaluation history

Date Model Scores Action Summary
qwen/qwen3.6-35b-a3b — — Merged changes that enable log verbosity configuration via the CRAFT_VERBOSITY_LEVEL environment variable, defaulting to BRIEF on invalid input. Approved by two reviewers, passed CI, and updated two files.
qwen3.6-35b-a3b-mtp-q6 — — Merged a fix allowing logging verbosity configuration via the CRAFT_VERBOSITY_LEVEL environment variable. Invalid inputs default to BRIEF. The change was authored by maintainer syu-w and successfully integrated.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#4699 fix(log): map SNAPCRAFT_VERBOSITY_LEVEL to CRAFT snapcraft merged Merged a fix mapping SNAPCRAFT_VERBOSITY_LEVEL to CRAFT logging. Approved by three reviewers with passing test coverage. The change modifies five files, adding 30 lines and removing 4.
76%
#3958 cli: set default verbosity level with environment variable snapcraft merged Merged. Adds SNAPCRAFT_VERBOSITY_LEVEL env var for default CLI verbosity. Author noted implementation flaws causing duplicate logs and precedence issues, opening craft-cli issue #128 for a proper fix.
75%
#94 Refactor how the logger Handler behaves for the new verbosity levels (CRAFT-1138). craft-cli merged Merged refactoring of the logger Handler to support new verbosity levels. Approved by one reviewer with zero unresolved comments. The change modified four files, adding 114 lines and removing 43.
73%
#264 Support CRAFT_VERBOSITY_LEVEL craft-application closed Added support for the CRAFT_VERBOSITY_LEVEL environment variable to configure initial verbosity, matching Snapcraft's implementation. The change was merged via pull request #289 in the craft-application repository.
72%
#801 Adapted project to the new Craft CLI verbosity levels (CRAFT-1143). charmcraft merged Merged after approvals and CI. Updates logging to debug, switches to progress/permanent messages, simplifies startup via --verbosity, initializes emit in BRIEF mode, and refreshes dependencies to match new Craft CLI verbosity levels.
72%
#88 Changed NORMAL to BRIEF and added a DEBUG verbosity level (still not used) (CRAFT-1134) craft-cli merged Merged following single reviewer approval. Replaced NORMAL verbosity with BRIEF and introduced an unused DEBUG level. Changes are strictly mechanical renames across eight files, preserving existing semantics.
70%