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

Log4JXmlEventLayout - Added IncludeCallSite + IncludeSourceInfo #3442

Merged

Conversation

snakefoot
Copy link
Contributor

@snakefoot snakefoot commented May 30, 2019

Missing a nail to resolve #3387

One could probably rebase to NLog:master

@SkinnySackboy
Copy link

Thanks for this.

@304NotModified 304NotModified added this to the 5.0 (new) milestone May 30, 2019
@304NotModified
Copy link
Member

One could probably rebase to NLog:master

I think that's better indeed. NLog 5 take some weeks at least

@codecov-io
Copy link

codecov-io commented May 30, 2019

Codecov Report

Merging #3442 into release/4.6.5 will increase coverage by <1%.
The diff coverage is 0%.

@@              Coverage Diff               @@
##           release/4.6.5   #3442    +/-   ##
==============================================
+ Coverage             80%     80%   +<1%     
==============================================
  Files                359     359            
  Lines              28634   28638     +4     
  Branches            3817    3817            
==============================================
+ Hits               22916   22931    +15     
+ Misses              4620    4604    -16     
- Partials            1098    1103     +5

@snakefoot snakefoot force-pushed the Log4JXmlEventLayoutCallSite branch from a1f75a8 to 4ed3aeb Compare May 30, 2019 10:27
@snakefoot snakefoot changed the base branch from dev to master May 30, 2019 10:27
@snakefoot snakefoot force-pushed the Log4JXmlEventLayoutCallSite branch from 4ed3aeb to 334e572 Compare May 30, 2019 10:28
@snakefoot
Copy link
Contributor Author

think that's better indeed. NLog 5 take some weeks at least

Rebased to master

@304NotModified 304NotModified changed the base branch from master to release/4.6.5 May 30, 2019 19:01
@304NotModified 304NotModified merged commit e858765 into NLog:release/4.6.5 May 30, 2019
@304NotModified
Copy link
Member

@snakefoot could you please help me with the docs?

Working on a release now

@snakefoot
Copy link
Contributor Author

Have updated the wiki: https://github.com/NLog/NLog/wiki/Log4JXmlEventLayout

@304NotModified
Copy link
Member

thanks!

@304NotModified 304NotModified added the documentation done all docs done (wiki, api docs, lists on nlog-project.org, xmldocs) label Jun 11, 2019
@304NotModified
Copy link
Member

FYI, delaying 4.6.5 for #3472 😞

@304NotModified
Copy link
Member

@SkinnySackboy NLog 4.6.5 has been released! https://www.nuget.org/packages/NLog/4.6.5 :)

@snakefoot snakefoot deleted the Log4JXmlEventLayoutCallSite branch April 4, 2020 12:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation done all docs done (wiki, api docs, lists on nlog-project.org, xmldocs) feature needs documentation on wiki
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants