Skip to content

ShortDateLayoutRenderer with support for IRawValue + IStringValueRenderer#3337

Merged
304NotModified merged 2 commits intoNLog:devfrom
snakefoot:ShortDateStringRender
Apr 27, 2019
Merged

ShortDateLayoutRenderer with support for IRawValue + IStringValueRenderer#3337
304NotModified merged 2 commits intoNLog:devfrom
snakefoot:ShortDateStringRender

Conversation

@snakefoot
Copy link
Copy Markdown
Contributor

@snakefoot snakefoot commented Apr 27, 2019

Makes it possible to get current date as raw-value (Without time-part)

Makes it possible to use ${ShortDate} in cacheKey for https://github.com/NLog/NLog/wiki/Cached-Layout-Renderer with reduced string-allocations.

@snakefoot snakefoot force-pushed the ShortDateStringRender branch from 3e95b3f to e66ffba Compare April 27, 2019 08:21
@codecov-io
Copy link
Copy Markdown

codecov-io commented Apr 27, 2019

Codecov Report

Merging #3337 into dev will decrease coverage by <1%.
The diff coverage is 100%.

@@          Coverage Diff           @@
##             dev   #3337    +/-   ##
======================================
- Coverage     80%     80%   -<1%     
======================================
  Files        356     356            
  Lines      28213   28226    +13     
  Branches    3763    3763            
======================================
+ Hits       22572   22579     +7     
- Misses      4555    4558     +3     
- Partials    1086    1089     +3

@304NotModified 304NotModified added the enhancement Improvement on existing feature label Apr 27, 2019
@304NotModified 304NotModified added this to the 4.6.3 milestone Apr 27, 2019
@304NotModified
Copy link
Copy Markdown
Member

Thanks!

@304NotModified 304NotModified merged commit 34fe8ae into NLog:dev Apr 27, 2019
@snakefoot snakefoot deleted the ShortDateStringRender branch April 4, 2020 12:56
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.

3 participants