Skip to content

Add dictConfig#61

Merged
tonybaloney merged 10 commits into
microsoft:mainfrom
aminalaee:add-dictconfig
Aug 31, 2022
Merged

Add dictConfig#61
tonybaloney merged 10 commits into
microsoft:mainfrom
aminalaee:add-dictconfig

Conversation

@aminalaee

@aminalaee aminalaee commented Aug 10, 2022

Copy link
Copy Markdown
Contributor

Fixes #53

Please review #63 before this.

@aminalaee aminalaee changed the title Add dictconfig Add dictConfig Aug 10, 2022
@tonybaloney

Copy link
Copy Markdown
Collaborator

please rebase

@codecov-commenter

codecov-commenter commented Aug 20, 2022

Copy link
Copy Markdown

Codecov Report

Merging #61 (b41f537) into main (1e1ffc5) will decrease coverage by 0.85%.
The diff coverage is 77.72%.

@@            Coverage Diff             @@
##             main      #61      +/-   ##
==========================================
- Coverage   86.31%   85.45%   -0.86%     
==========================================
  Files          10       11       +1     
  Lines        1900     2111     +211     
==========================================
+ Hits         1640     1804     +164     
- Misses        260      307      +47     
Impacted Files Coverage Δ
src/picologging/config.py 77.72% <77.72%> (ø)
src/picologging/handlers.py 93.82% <0.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@aminalaee
aminalaee marked this pull request as ready for review August 20, 2022 17:47
Comment thread src/picologging/__init__.py Outdated
@tonybaloney
tonybaloney merged commit 0a9c68b into microsoft:main Aug 31, 2022
@tonybaloney

Copy link
Copy Markdown
Collaborator

Thanks I'll ship this after reviewing the others

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add support for dictConfig

3 participants