Conversation
📝 WalkthroughWalkthroughThe README "From source" Quick Start installation command now uses ChangesQuick Start Installation Documentation
Estimated code review effort: 1 (Trivial) | ~2 minutes Possibly related PRs
Suggested reviewers: 🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 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 |
There was a problem hiding this comment.
Pull request overview
Note
Copilot was unable to run its full agentic suite in this review.
Updates the installation instructions to install the CLI skill from the local repository instead of from the larksuite/cli package reference.
Changes:
- Replace
npx skills add larksuite/cli -y -gwithnpx skills add ./ -y -gin the README install instructions.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Summary: Update README.md to instruct users to install the CLI SKILL from the local clone instead of from the GitHub repository. If users install from the GitHub repository, they may not get the latest changes that are present in the local clone.
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.
Suppressed comments (1)
README.md:79
- The switch to a local-path install (
./) is only implied by the command. Since other user-facing docs/help still recommendnpx skills add larksuite/cli -y -g(e.g. README.zh.md and the root help template), this section can be confusing. Suggest making the “from source” instructions explicitly say the skills are installed from the local clone, and consider aligning the other references in a follow-up.
npx skills add ./ -y -g
PR Quality SummaryCI did not complete successfully. Use the failed check links below to decide whether this PR needs a code change or a rerun. CI status
|
Summary: Update README.md to instruct users to install the CLI SKILL from the local clone instead of from the GitHub repository. If users install from the GitHub repository, they may not get the latest changes that are present in the local clone.
Summary by CodeRabbit