-
Notifications
You must be signed in to change notification settings - Fork 1.4k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
NLog config - Fixed bug where empty xml-elements were ignored #3271
NLog config - Fixed bug where empty xml-elements were ignored #3271
Conversation
…t Value or attributes was ignored
great work! |
Would have been even greater if the bug was never made. Especially since I created it :) But we now have an unit-test to protect us from it happening again. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍 thanks
Codecov Report
@@ Coverage Diff @@
## dev #3271 +/- ##
======================================
+ Coverage 80% 80% +<1%
======================================
Files 355 355
Lines 27986 27995 +9
Branches 3727 3728 +1
======================================
+ Hits 22324 22358 +34
+ Misses 4579 4554 -25
Partials 1083 1083 |
congrats @snakefoot |
Yes why not make it "Champagne Tuesday" :). And thank you |
🍾 🎉 💥 |
LoggingConfigurationParser - Fixed bug where empty xml-node-elements was ignored
Resolves #3268