Skip to content
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

EventPropertiesLayoutRenderer - Support Json Format #2241

Merged
merged 2 commits into from
Aug 13, 2017

Conversation

snakefoot
Copy link
Contributor

@snakefoot snakefoot commented Aug 4, 2017

Guess it makes most sense when combined with #2208 (Or #2244)

@codecov
Copy link

codecov bot commented Aug 4, 2017

Codecov Report

Merging #2241 into master will decrease coverage by <1%.
The diff coverage is 92%.

@@           Coverage Diff           @@
##           master   #2241    +/-   ##
=======================================
- Coverage      82%     82%   -<1%     
=======================================
  Files         304     304            
  Lines       21126   21132     +6     
  Branches     2531    2532     +1     
=======================================
+ Hits        17275   17279     +4     
- Misses       3216    3217     +1     
- Partials      635     636     +1

@codecov
Copy link

codecov bot commented Aug 4, 2017

Codecov Report

Merging #2241 into master will increase coverage by <1%.
The diff coverage is 93%.

@@           Coverage Diff           @@
##           master   #2241    +/-   ##
=======================================
+ Coverage      82%     82%   +<1%     
=======================================
  Files         304     304            
  Lines       21126   21127     +1     
  Branches     2531    2531            
=======================================
+ Hits        17275   17277     +2     
+ Misses       3216    3212     -4     
- Partials      635     638     +3

@304NotModified 304NotModified added this to the 4.5 milestone Aug 13, 2017
@304NotModified
Copy link
Member

Updated the docs, feel free to change.

@codecov-io
Copy link

Codecov Report

Merging #2241 into master will decrease coverage by <1%.
The diff coverage is 93%.

@@           Coverage Diff           @@
##           master   #2241    +/-   ##
=======================================
- Coverage      82%     82%   -<1%     
=======================================
  Files         304     304            
  Lines       21126   21312   +186     
  Branches     2531    2570    +39     
=======================================
+ Hits        17275   17393   +118     
- Misses       3216    3263    +47     
- Partials      635     656    +21

@304NotModified 304NotModified merged commit 0b2d657 into NLog:master Aug 13, 2017
@304NotModified 304NotModified modified the milestones: 4.5, 4.5 beta 1 Sep 30, 2017
@snakefoot snakefoot mentioned this pull request Nov 23, 2017
@snakefoot snakefoot deleted the EventPropertiesJsonFormat branch January 29, 2018 18:11
@snakefoot snakefoot modified the milestones: 4.5 beta 1, 4.5 Aug 22, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants