Skip to content

feat: add delimiter for fileName depth to %f layout patern#913

Merged
nomiddlename merged 2 commits intolog4js-node:masterfrom
rommni:feature/fileNameTruncate
Jul 9, 2019
Merged

feat: add delimiter for fileName depth to %f layout patern#913
nomiddlename merged 2 commits intolog4js-node:masterfrom
rommni:feature/fileNameTruncate

Conversation

@rommni
Copy link
Copy Markdown

@rommni rommni commented Jul 9, 2019

solved #888 without using the token way, that i personnally found less clear.

@codecov
Copy link
Copy Markdown

codecov Bot commented Jul 9, 2019

Codecov Report

Merging #913 into master will increase coverage by 0.01%.
The diff coverage is 100%.

Impacted file tree graph

@@           Coverage Diff            @@
##           master   #913      +/-   ##
========================================
+ Coverage   97.98%    98%   +0.01%     
========================================
  Files          25     25              
  Lines         995   1002       +7     
========================================
+ Hits          975    982       +7     
  Misses         20     20
Impacted Files Coverage Δ
lib/layouts.js 96.26% <100%> (+0.26%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 0d7fd0c...4bfbe7c. Read the comment docs.

Copy link
Copy Markdown
Collaborator

@nomiddlename nomiddlename left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for this. test + docs = happy maintainer.

@nomiddlename nomiddlename added this to the 4.4.1 milestone Jul 9, 2019
@nomiddlename nomiddlename merged commit f428d7d into log4js-node:master Jul 9, 2019
@nomiddlename
Copy link
Copy Markdown
Collaborator

Published to npm in [email protected]

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants