← Back to issue list

fix(log): map SNAPCRAFT_VERBOSITY_LEVEL to CRAFT

View original Github issue

Metadata

Project
snapcraft
Number
#4699
Type
pull request
State
merged
Author
syu-w
Labels
Created
Updated
Closed

Current evaluation

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.

Suggested action:

No scores available.

Issue body

No description provided.

Evaluation history

Date Model Scores Action Summary
qwen/qwen3.6-35b-a3b 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.
qwen3.6-35b-a3b-mtp-q6 Merged a fix mapping SNAPCRAFT_VERBOSITY_LEVEL to CRAFT. The change passed all tests with full coverage on modified lines, raising overall project coverage to 89.56%.
qwen3.6-35b-a3b-mtp-q6 Merged a fix mapping SNAPCRAFT_VERBOSITY_LEVEL to CRAFT logging configuration. Codecov confirmed full test coverage on modified lines, raising overall project coverage to 89.56%.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#289 fix(log): allows set logging verbosity level from env craft-application merged 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.
76%
#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.
71%
#800 many: reword docstrings and code comments charmcraft merged Merged following single reviewer approval. Reworded docstrings and code comments across five files to improve clarity, resolving CRAFT-697. The +10/-10 line change passed all CI checks.
71%
#1996 tests: add SNAPCRAFT_KEEP_DATA to debug snapcraft merged Merged test updates adding the SNAPCRAFT_KEEP_DATA environment variable for debugging. Approved by two reviewers, the change modifies two files with a net addition of 14 lines. Submitted as a farewell gift to @sergiusens.
70%
#6098 fix: Tweak error message about SNAPCRAFT_STORE_CREDENTIALS snapcraft merged Merged to clarify the ambiguous SNAPCRAFT_STORE_CREDENTIALS error message. The update explicitly states login fails while the variable is set, resolving user confusion. Approved by four reviewers and passed CI checks prior to merge.
70%