openvax / gtfparse / 28953968093

Build # 28953968093

Build Type

Pull #71

github

Committed by iskandr

Commit Message

Fix #70: stop calling logging.basicConfig() at import time

gtfparse called logging.basicConfig(level=logging.INFO) at module import
in three modules (read_gtf, attribute_parsing, create_missing_features).
As a library this is an anti-pattern: it configures the root logger on
behalf of the importing application, adding a handler and forcing the
INFO level onto everyone's root logger — hijacking the host app's logging.

Remove the basicConfig() calls and route the remaining log statements
through the module-level `logger` (logging.getLogger(__name__)) instead
of the root-logger convenience functions logging.info(...), which would
otherwise implicitly re-trigger basicConfig().

Configuring handlers/levels is now left entirely to the application, per
the standard library logging guidance for libraries.

Claude-Session: https://claude.ai/code/session_014fNxSBm5zvdsDNwN3nhmpS

Pull Request Pull Request #71: Fix #70: stop calling logging.basicConfig() at import time

Coverage Stats

196 of 206 relevant lines covered (95.15%)

0.95 hits per line

Source File

Press 'n' to go to next uncovered line, 'b' for previous

Source Not Available