Skip to content

feat!: remove STDIN template; take template as positional arg#942

Open
harttle wants to merge 2 commits into
nextfrom
fix/remove-stdin-template
Open

feat!: remove STDIN template; take template as positional arg#942
harttle wants to merge 2 commits into
nextfrom
fix/remove-stdin-template

Conversation

@harttle

@harttle harttle commented Jul 23, 2026

Copy link
Copy Markdown
Owner

Summary

Fixes #940

Test plan

  • npx liquidjs ''{{ hello | capitalize }}'' renders
  • npx liquidjs @./file.liquid reads template from file
  • echo ''{name:Snake}'' | npx liquidjs ''Hello, {{ name }}!'' --context @- works
  • Piped stdin without --context @- is ignored
  • npx liquidjs --template ... fails (unknown option)
  • Docs no longer use --template or template-via-stdin

Fixes #940

Co-authored-by: Cursor <cursoragent@cursor.com>
@coveralls

coveralls commented Jul 23, 2026

Copy link
Copy Markdown

Coverage Report for CI Build 30032482860

Warning

No base build found for commit 964a63b on next.
Coverage changes can't be calculated without a base build.
If a base build is processing, this comment will update automatically when it completes.

Coverage: 99.528%

Details

  • Patch coverage: No coverable lines changed in this PR.

Uncovered Changes

No uncovered changes found.

Coverage Regressions

Requires a base build to compare against. How to fix this →


Coverage Stats

Coverage Status
Relevant Lines: 2973
Covered Lines: 2964
Line Coverage: 99.7%
Relevant Branches: 1052
Covered Branches: 1042
Branch Coverage: 99.05%
Branches in Coverage %: Yes
Coverage Strength: 18997.06 hits per line

💛 - Coveralls

Make template a required positional arg (drop --template) for v11 per #586; stdin template remains unsupported without a special error.

Co-authored-by: Cursor <cursoragent@cursor.com>
@harttle harttle changed the title feat!: remove CLI support for template via STDIN feat!: remove STDIN template; take template as positional arg Jul 23, 2026
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