Skip to content

fix: add trailing line feed to formatted JSON - #623

Merged
targos merged 1 commit into
nodejs:mainfrom
tniessen:json-lf
Apr 25, 2022
Merged

fix: add trailing line feed to formatted JSON#623
targos merged 1 commit into
nodejs:mainfrom
tniessen:json-lf

Conversation

@tniessen

Copy link
Copy Markdown
Member

Pretty-printed JSON files should end with \n.

@codecov

codecovBot commented Apr 21, 2022

Copy link
Copy Markdown

Codecov Report

Merging #623 (5152e8d) into main (649a3b0) will not change coverage.
The diff coverage is 100.00%.

@@ Coverage Diff @@## main #623 +/- ##
=======================================
Coverage 84.13% 84.13% =======================================
Files 37 37 Lines 4065 4065 =======================================
Hits 3420 3420 Misses 645 645 
Impacted FilesCoverage Δ
lib/file.js79.48% <100.00%> (ø)

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 649a3b0...5152e8d. Read the comment docs.

@tniessentniessen added the enhancement Things that enhances functionality, provided by node-core-utils label Apr 21, 2022
@targos
targos merged commit 1bcc72b into nodejs:mainApr 25, 2022
@github-actionsgithub-actionsBot mentioned this pull request Apr 25, 2022
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancementThings that enhances functionality, provided by node-core-utils

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@tniessen@targos