docs: clarify base percentage number values#1575
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (2)
📝 WalkthroughWalkthroughThe PR updates Lark Base reference docs to state that ChangesLark Base percentage docs
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ 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 |
f36da9d to
805c125
Compare
🚀 PR Preview Install Guide🧰 CLI updatenpm i -g https://pkg.pr.new/larksuite/cli/@larksuite/cli@805c125112519b93ce3065dd28fdeef9f3276b64🧩 Skill updatenpx skills add zgz2048/cli#codex/base-percentage-docs -y -g |
Summary
Clarifies that Base percentage number formatting only affects frontend display. Agents should write the underlying numeric value for CellValue and field default_value instead of the displayed percentage text.
Changes
style.percentage=truesemantics in Base CellValue guidance.Test Plan
lark-cli <domain> <command>flow works as expectedgit diff --checkRelated Issues
Summary by CodeRabbit