Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly
Sign up
Appearance settings

doc: tell users to install skills from local clone#1425

Open
vegerot wants to merge 1 commit into
larksuite:main from
vegerot:pr1425
Open

doc: tell users to install skills from local clone #1425
vegerot wants to merge 1 commit into
larksuite:main from
vegerot:pr1425

Conversation

@vegerot

@vegerot vegerot commented Jun 12, 2026 •
edited by coderabbitai Bot
Loading

Copy link
Copy Markdown

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

  • Documentation
    • Updated README Quick Start to install the CLI skill from the current directory using npx skills add ./ -y -g instead of the previous package path, clarifying how to build/install from source.

Copilot AI review requested due to automatic review settings June 12, 2026 03:41

coderabbitai Bot commented Jun 12, 2026 •
edited
Loading

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

i️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 6dcbcffc-34f9-4f3a-9334-f8da06315c83

📥 Commits

Reviewing files that changed from the base of the PR and between ac51ac3 and f0ed507.

📒 Files selected for processing (1)
  • README.md
✅ Files skipped from review due to trivial changes (1)
  • README.md
📝 Walkthrough

Walkthrough

The README "From source" Quick Start installation section now uses npx skills add ./ -y -g to install the CLI SKILL from the current directory instead of npx skills add larksuite/cli -y -g.

Changes

Quick Start Installation Documentation

Layer / File(s) Summary
Installation command update
README.md		 The CLI SKILL installation command in the "From source" section is changed to use local directory (./) instead of the larksuite/cli package path.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Suggested reviewers

  • liangshuo-1

Poem

🐰 A hop, a skip, a dot-slash cheer,
From source the CLI now draws near—
Install local, test and play,
The bunny guides the developer's way,
Tiny change, a brighter day.

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Description check ⚠️ Warning The description is incomplete. It lacks required sections like Changes, Test Plan, and Related Issues specified in the template. Add the missing sections from the template: Changes (listing the specific README updates), Test Plan (verification steps), and Related Issues sections to match the repository's description standard.
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately describes the main change: updating documentation to instruct users to install CLI SKILL from local clone instead of GitHub repository.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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 -g with npx skills add ./ -y -g in the README install instructions.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread README.md Outdated
@github-actions github-actions Bot added the size/S Low-risk docs, CI, test, or chore only changes label Jun 12, 2026

CLAassistant commented Jun 12, 2026 •
edited
Loading

Copy link
Copy Markdown

CLA assistant check
All committers have signed the CLA.

 

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.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

Copilot code review Copilot Copilot left review comments

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

size/S Low-risk docs, CI, test, or chore only changes

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

AltStyle によって変換されたページ (->オリジナル) /