• Home
  • Features
  • Pricing
  • Docs
  • Announcements
  • Sign In

osteele / liquid / 149 / 1
49%
master: 49%

Build:
Build:
LAST BUILD BRANCH: sgargan/whitespace-handling
DEFAULT BRANCH: master
Ran 20 Apr 2019 12:20PM UTC
Files 50
Run time 2s
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst

20 Apr 2019 12:18PM UTC coverage: 89.392% (+0.2%) from 89.183%
149.1

Pull #36

travis-ci

web-flow
Modify whitespace trimming to be less hungry

The liquid spec is a little vague about quite how greedy whitespace
trimming should be. Currently handling here will eat any available
whitespace between tags including all newline characters
e.g.
```
(

{%- sometag -%}

)
```
will render as
```
()
```
This is ok for web markup where the whitespace is not important, but for
other document types this greedy consumption is problematic.

Ideally the ws trimming would be less hungry and only trim any whitespace
on the line containing the tag up to and including it's newline, essentially
removing any trace of the tag but leaving other lines intact rendering the above as
```
(

)
```

The change here detects whitespace as tokens as they are parsed and
discards any as indicated by the trimming directives up to an including
the first newline.
Pull Request #36: Modify whitespace trimming to be less hungry

2663 of 2979 relevant lines covered (89.39%)

611.81 hits per line

Source Files on job 149.1
  • Tree
  • List 0
  • Changed 31
  • Source Changed 6
  • Coverage Changed 30
Coverage ∆ File Lines Relevant Covered Missed Hits/Line
  • Back to Build 146
  • Travis Job 149.1
  • 10b002ef on github
  • Prev Job for on sgargan/whitespace-handling (#143.1)
STATUS · Troubleshooting · Open an Issue · Sales · Support · CAREERS · ENTERPRISE · START FREE · SCHEDULE DEMO
ANNOUNCEMENTS · TWITTER · TOS & SLA · Supported CI Services · What's a CI service? · Automated Testing

© 2026 Coveralls, Inc