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

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

Build:
Build:
LAST BUILD BRANCH: sgargan/whitespace-handling
DEFAULT BRANCH: master
Ran 19 Apr 2019 11:42PM UTC
Files 50
Run time 3s
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

19 Apr 2019 11:40PM UTC coverage: 89.056% (-0.1%) from 89.183%
148.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

2661 of 2988 relevant lines covered (89.06%)

609.95 hits per line

Source Files on job 148.1
  • Tree
  • List 0
  • Changed 29
  • Source Changed 6
  • Coverage Changed 28
Coverage ∆ File Lines Relevant Covered Missed Hits/Line
  • Back to Build 145
  • Travis Job 148.1
  • 321cd3ed 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