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

Changed IIncludeContext to internal interface until someone needs it #2421

Merged
merged 1 commit into from
Nov 26, 2017

Conversation

snakefoot
Copy link
Contributor

@snakefoot snakefoot commented Nov 26, 2017

Minor fix to #2117

@codecov
Copy link

codecov bot commented Nov 26, 2017

Codecov Report

Merging #2421 into master will increase coverage by <1%.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master   #2421    +/-   ##
=======================================
+ Coverage      82%     82%   +<1%     
=======================================
  Files         322     322            
  Lines       23020   23020            
  Branches     2846    2846            
=======================================
+ Hits        18785   18789     +4     
+ Misses       3504    3499     -5     
- Partials      731     732     +1

@304NotModified
Copy link
Member

Good idea +1

@304NotModified 304NotModified merged commit b57204c into NLog:master Nov 26, 2017
@304NotModified 304NotModified added this to the 4.5 rc2 milestone Nov 26, 2017
@304NotModified 304NotModified added the enhancement Improvement on existing feature label Nov 26, 2017
@snakefoot snakefoot deleted the IIncludeContextInternal branch December 4, 2017 19:48
@snakefoot snakefoot modified the milestones: 4.5 rc2, 4.5 Aug 22, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Improvement on existing feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants