Repository navigation
chore(compilers): upgrade liquidjs to v10.30.0 - #1078
Conversation
✅ Deploy Preview for livecodes ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
|
|
No actionable comments were generated in the recent review.
 |
| Layer / File(s) | Summary |
|---|---|
Update LiquidJS version references src/livecodes/vendors.ts, docs/docs/languages/liquid.mdx |
The vendor URL and language documentation now reference LiquidJS 10.30.0. |
Priority:
Estimated code review effort: 1 (Trivial) | ~3 minutes
Change: Other
Suggested reviewers: hatemhosny
Merge Risk: 
20f0c
-
No actionable merge-blocking risk remains.
Pre-merge checks: 
-
Check name Status Explanation Description Check Check skipped - CodeRabbit’s high-level summary is enabled. Title check The title clearly and concisely describes the main change: upgrading LiquidJS from v10.14.0 to v10.30.0. Docstring Coverage No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 1… Linked Issues check Check skipped because no linked issues were found for this pull request. Out of Scope Changes check Check skipped because no linked issues were found for this pull request. Full details: Docstring Coverage
-
Explanation
No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 1 files. (1 skipped: 1 unsupported.)
-
Finishing Touches
-
🧪 Generate unit tests (beta)
-
- Create a new PR
-
- Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.
Comment @coderabbitai help to get the list of available commands.
![pullfrog[bot]](https://avatars.githubusercontent.com/in/1768019?s=60&v=4){kind=small}
There was a problem hiding this comment.
✅ No new issues found.
Reviewed changes
- Liquid compiler version bump —
liquidJsUrlinsrc/livecodes/vendors.ts:338moved fromliquidjs@10.14.0toliquidjs@10.30.0(CDN URL unchanged otherwise). - Documentation sync —
docs/docs/languages/liquid.mdx:77updated to documentv10.30.0.
Verified during review:
liquidjs@10.30.0exists on the npm registry and still shipsdist/liquid.browser.min.js, so the CDN URL resolves.- The 10.30.0 UMD bundle still exposes the global
liquidjsand aLiquidconstructor withparseAndRender, solang-liquid-compiler.tsandlang-liquid.tskeep working unchanged. - No other references to the previous version remain.
deepseek-v4.1-flash (free via Pullfrog for OSS) ｜ 𝕏
|
Thank you @MohamedAbdElwahabOka |
What type of PR is this? (check all applicable)
Description
This PR upgrades the Liquid compiler (
liquidjs) from v10.14.0 to v10.30.0 (the latest release, same major version, so no breaking changes are expected).Changes:
src/livecodes/vendors.ts: bumpliquidJsUrltoliquidjs@10.30.0docs/docs/languages/liquid.mdx: update the documented versionRelated Tickets & Documents
Part of #875
Mobile & Desktop Screenshots/Recordings
There is no Liquid starter template, so I tested locally by passing Liquid code in the URL:
http://127.0.0.1:8080/?liquid={% assign name = "world" %}<h1>Hello {{ name | upcase }}</h1>{% for i in (1..3) %}<p>Item {{ i }}</p>{% endfor %}assign, theupcasefilter and theforloop all render correctly, and there are no console errors:Added tests?
Added to documentations?
Summary by CodeRabbit