docs(slides): use built-in --jq in cli examples and fix workflow link labels - #2286
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (3)
🚧 Files skipped from review as they are similar to previous changes (2)
📝 WalkthroughWalkthroughThe Slides CLI documentation now uses built-in ChangesSlides CLI documentation
Estimated code review effort: 1 (Trivial) | ~5 minutes Possibly related PRs
Suggested reviewers: 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
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 |
🚀 PR Preview Install Guide🧰 CLI updatenpm i -g https://pkg.pr.new/larksuite/cli/@larksuite/cli@3abd20a8d9e659607668076cd69f028db47a06f8🧩 Skill updatenpx skills add larksuite/cli#apply-slides-cli-doc-tweaks -y -g |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #2286 +/- ##
==========================================
+ Coverage 76.44% 76.46% +0.01%
==========================================
Files 1013 1013
Lines 111972 112034 +62
==========================================
+ Hits 85601 85665 +64
+ Misses 19860 19857 -3
- Partials 6511 6512 +1 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
Small docs-only fixes to the lark-slides
references/cli/command references. No code changed.Summary
Update 4 CLI command docs: replace the external
| jq -rpipe in examples with lark-cli's built-in--jqflag, and fix stale workflow link labels. Applied from local edits in the internal mirror.Changes
lark-slides-media-upload.md—| jq -r '.data.file_token'→--jq '.data.file_token'; drop a stray leading blank line.lark-slides-replace-slide.md— same--jqswap; link labellark-slides-edit-workflows.md→slides_editing.md.lark-slides-xml-presentation-slide-get.md— link label →slides_editing.md.lark-slides-xml-presentation-slide-replace.md—--jqswap; link label →slides_editing.md.Why
--jq: it's a built-in lark-cli flag (-q/--jq, added in #211) that filters JSON output in-process, so examples no longer depend on an externaljqbinary being installed and avoid stderr progress lines polluting$(...).Summary by CodeRabbit
--jqoption, eliminating the need for external JSON-processing commands.