search

PyUtilib / pyutilib

A collection of general Python utilities, including logging and file IO, subprocess management, plugin systems, and workflow management.
BSD 3-Clause "New" or "Revised" License
34 stars 20 forks source link

Log formatter should not combine lines starting "[\s*\S+\s*]" #102

Closed jsiirola closed 4 years ago

jsiirola commented 4 years ago

Fixes: #N/A

Summary/Motivation:

This prevents the log formatter from combining lines that begin with "\[s*\S+\s*\]", so that log messages like the following are not reformatted:

INFO: The following extensions were downloaded:
        [FAIL]  gsl
        [ OK ]  mcpp

Changes proposed in this PR:

Legal Acknowledgement

By contributing to this software project, I agree to the following terms and conditions for my contribution:

  1. I agree my contributions are submitted under the BSD license.
  2. I represent I am authorized to make the contributions and grant the license. If my employer has rights to intellectual property that includes these contributions, I represent that I have received permission to make contributions and grant the required license on behalf of that employer.
coveralls commented 4 years ago

Coverage Status

Coverage increased (+0.004%) to 61.923% when pulling ac33f237463dfab6d324d33ac6f7c3f22a8c4683 on log-formatting into c0b323e2841b82f4585ebc6089fb5fd6ddb1ac48 on master.

codecov-commenter commented 4 years ago

Codecov Report

Merging #102 into master will increase coverage by 0.00%. The diff coverage is 100.00%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #102   +/-   ##
=======================================
  Coverage   63.96%   63.96%           
=======================================
  Files          87       87           
  Lines        8933     8934    +1     
=======================================
+ Hits         5714     5715    +1     
  Misses       3219     3219
Impacted Files Coverage Δ
pyutilib/misc/log_config.py 97.36% <100.00%> (+0.03%) :arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update c0b323e...ac33f23. Read the comment docs.