Skip to content

Commit 042862b

Browse files
Merge pull request #56 from MrMino/fix_debug_logging
Set log level globally, not just in the script
2 parents ac5a463 + da21daa commit 042862b

File tree

3 files changed

+11
-6
lines changed

3 files changed

+11
-6
lines changed

CHANGELOG.rst

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -7,6 +7,7 @@ Release History
77
- Added `--requirements-file` flag to `pip-extra-reqs` and `pip-missing-reqs`
88
commands. This flag makes it possible to specify a path to the requirements
99
file. Previously, `"requirements.txt"` was always used.
10+
- Fixed some of the logs not being visible with `-d` and `-v` flags.
1011

1112
2.1.1
1213

pip_check_reqs/find_extra_reqs.py

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -122,11 +122,13 @@ def main():
122122

123123
logging.basicConfig(format='%(message)s')
124124
if options.debug:
125-
log.setLevel(logging.DEBUG)
125+
level = logging.DEBUG
126126
elif options.verbose:
127-
log.setLevel(logging.INFO)
127+
level = logging.INFO
128128
else:
129-
log.setLevel(logging.WARN)
129+
level = logging.WARN
130+
log.setLevel(level)
131+
common.log.setLevel(level)
130132

131133
log.info('using pip_check_reqs-%s from %s', __version__, __file__)
132134

pip_check_reqs/find_missing_reqs.py

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -136,11 +136,13 @@ def main():
136136

137137
logging.basicConfig(format='%(message)s')
138138
if options.debug:
139-
log.setLevel(logging.DEBUG)
139+
level = logging.DEBUG
140140
elif options.verbose:
141-
log.setLevel(logging.INFO)
141+
level = logging.INFO
142142
else:
143-
log.setLevel(logging.WARN)
143+
level = logging.WARN
144+
log.setLevel(level)
145+
common.log.setLevel(level)
144146

145147
log.info('using pip_check_reqs-%s from %s', __version__, __file__)
146148

0 commit comments

Comments
 (0)