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

Use string interpolation #2337

Merged
merged 2 commits into from
Oct 12, 2017
Merged

Use string interpolation #2337

merged 2 commits into from
Oct 12, 2017

Conversation

c0shea
Copy link
Contributor

@c0shea c0shea commented Oct 11, 2017

Use C# 6 string interpolation instead of string.Format

@codecov
Copy link

codecov bot commented Oct 11, 2017

Codecov Report

Merging #2337 into master will not change coverage.
The diff coverage is 79%.

@@          Coverage Diff           @@
##           master   #2337   +/-   ##
======================================
  Coverage      82%     82%           
======================================
  Files         317     317           
  Lines       22716   22716           
  Branches     2784    2784           
======================================
  Hits        18609   18609           
  Misses       3405    3405           
  Partials      702     702

@304NotModified
Copy link
Member

1 conflict, please fix, thx!

@304NotModified 304NotModified added this to the 4.5 milestone Oct 11, 2017
@c0shea
Copy link
Contributor Author

c0shea commented Oct 12, 2017

Resolved the conflict

@304NotModified 304NotModified merged commit 643e326 into NLog:master Oct 12, 2017
@304NotModified
Copy link
Member

thanks!

@304NotModified 304NotModified modified the milestones: 4.5, 4.5 beta 5 Oct 12, 2017
@snakefoot snakefoot modified the milestones: 4.5 beta 5, 4.5 Aug 22, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants