Closed fatfisz closed 6 years ago
Merging #165 into master will not change coverage. The diff coverage is
100%
.
@@ Coverage Diff @@
## master #165 +/- ##
=====================================
Coverage 100% 100%
=====================================
Files 27 27
Lines 111 112 +1
Branches 29 29
=====================================
+ Hits 111 112 +1
Impacted Files | Coverage Δ | |
---|---|---|
src/stripIndent/stripIndent.js | 100% <ø> (ø) |
:arrow_up: |
src/commaListsOr/commaListsOr.js | 100% <ø> (ø) |
:arrow_up: |
src/commaLists/commaLists.js | 100% <ø> (ø) |
:arrow_up: |
src/commaListsAnd/commaListsAnd.js | 100% <ø> (ø) |
:arrow_up: |
src/html/html.js | 100% <ø> (ø) |
:arrow_up: |
src/stripIndents/stripIndents.js | 100% <ø> (ø) |
:arrow_up: |
src/safeHtml/safeHtml.js | 100% <ø> (ø) |
:arrow_up: |
src/testUtils/readFromFixture/readFromFixture.js | 100% <100%> (ø) |
:arrow_up: |
src/trimResultTransformer/trimResultTransformer.js | 100% <100%> (ø) |
:arrow_up: |
src/inlineLists/inlineLists.js | 100% <100%> (ø) |
:arrow_up: |
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 e393195...1b42a87. Read the comment docs.
This should fix the long-standing issue #37.
The rules for "smart trimming" are as such:
In this case it's safe to assume that the first newline is not wanted in the end result.
To sum up, smart trimming is about sane defaults and working together with indent-stripping tags.