openvax / gtfparse / 28965821496

Build # 28965821496

Build Type

Pull #73

github

Committed by iskandr

Commit Message

Fix #70: don't reconfigure root logging when importing gtfparse

attribute_parsing, create_missing_features and read_gtf each called
logging.basicConfig(level=logging.INFO) at import, which adds a handler to the
ROOT logger and sets the root level to INFO — hijacking downstream applications'
logging configuration and breaking pytest's caplog in their test suites.

Per the Python logging HOWTO ("Configuring Logging for a Library"), remove the
import-time basicConfig calls and attach a NullHandler to the top-level gtfparse
logger instead, so importing gtfparse is side-effect-free. Applications opt in
to gtfparse's INFO logs via their own logging configuration.

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

Pull Request Pull Request #73: Fix #70: don't call logging.basicConfig() at import

Coverage Stats

198 of 208 relevant lines covered (95.19%)

0.95 hits per line

Source File

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

Source Not Available