Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Add support for passing LiquidOptions from CLI #583

Closed
DaRosenberg opened this issue Feb 7, 2023 · 2 comments · Fixed by #586
Closed

Add support for passing LiquidOptions from CLI #583

DaRosenberg opened this issue Feb 7, 2023 · 2 comments · Fixed by #586
Labels
feature requested released

Comments

@DaRosenberg
Copy link
Contributor

It would be very useful to be able to pass options from the CLI, for example:

cat template.liquid | npx liquidjs --jsTruthy=true '{"name": "Snake"}'

If idea is supported by maintainers, I will contribute this as a PR.
This was referenced Feb 7, 2023
Closed
Closed
@harttle
Copy link
Owner

harttle commented Feb 7, 2023

I'm glad that finally we have a CLI related issue. It makes sense to allow passing options. I can help review and publish your change.

@DaRosenberg DaRosenberg mentioned this issue Feb 8, 2023
Merged
@github-actions
Copy link

🎉 This issue has been resolved in version 10.5.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
feature requested released
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Adds support for options to CLI and improves usability DaRosenberg/liquidjs
2 participants
@DaRosenberg @harttle