docs: Update dingtalk QR code and default port for docker#681
docs: Update dingtalk QR code and default port for docker#681xieyxclack merged 2 commits intomainfrom
Conversation
|
Warning You have reached your daily quota limit. Please wait up to 24 hours and I will start processing your requests again! |
📝 WalkthroughWalkthroughUpdates Docker run examples to bind container ports to localhost (127.0.0.1) across docs and code; updates DingTalk/Discord badge image URLs and dimensions; refines the PR-label GitHub Actions workflow with guarded first-time-contributor checks and error handling. Changes
Sequence Diagram(s)(omitted) Estimated code review effort🎯 3 (Moderate) | ⏱️ ~25 minutes Possibly related PRs
Suggested reviewers
Poem
🚥 Pre-merge checks | ✅ 2 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (2 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 |
There was a problem hiding this comment.
Pull request overview
This PR updates website/docs content by (1) refreshing the DingTalk community QR code image and (2) changing Docker run examples to bind the exposed port to localhost (127.0.0.1) for safer defaults when running locally.
Changes:
- Update Docker run commands in Quick Start/FAQ/README/docs to use
-p 127.0.0.1:8088:8088. - Update DingTalk QR code image URL in community pages and READMEs.
- Update
scripts/docker_build.shprinted run instructions to match the localhost-only port binding.
Reviewed changes
Copilot reviewed 11 out of 11 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| website/src/components/QuickStart.tsx | Updates Docker quickstart command shown in the UI to bind port to localhost. |
| website/public/docs/quickstart.zh.md | Updates Docker quickstart command to bind port to localhost. |
| website/public/docs/quickstart.en.md | Updates Docker quickstart command to bind port to localhost. |
| website/public/docs/faq.zh.md | Updates Docker commands in FAQ to bind port to localhost. |
| website/public/docs/faq.en.md | Updates Docker commands in FAQ to bind port to localhost. |
| website/public/docs/community.zh.md | Updates DingTalk QR image URL in community table. |
| website/public/docs/community.en.md | Updates DingTalk QR image URL in community table. |
| scripts/docker_build.sh | Updates printed docker run examples to bind ports to localhost. |
| README_zh.md | Updates Docker command and DingTalk QR image URL. |
| README_ja.md | Updates Docker command and DingTalk QR image URL. |
| README.md | Updates Docker command and DingTalk QR image URL. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In @.github/workflows/pr-label.yml:
- Around line 22-28: The search-based first-time check using
github.rest.search.issuesAndPullRequests is unreliable; replace it by listing
repo PRs with the pulls API and checking the PR author directly. Call
github.rest.pulls.list({ owner: context.repo.owner, repo: context.repo.repo,
state: 'all', per_page: 100 }) (with pagination if needed), then compute
isFirstTime by ensuring no returned PR has pr.user.login === author (or count
matching PRs); update the code paths that set isFirstTime and the logging (the
variables/functions to change are isFirstTime and the block using
github.rest.search.issuesAndPullRequests).
ℹ️ Review info
⚙️ Run configuration
Configuration used: defaults
Review profile: CHILL
Plan: Pro
Run ID: 22a6b909-fc98-49df-8127-b5e224cfcbd8
📒 Files selected for processing (1)
.github/workflows/pr-label.yml
Description
[Describe what this PR does and why]
Related Issue: Fixes #(issue_number) or Relates to #(issue_number)
Security Considerations: [If applicable, e.g. channel auth, env/config handling]
Type of Change
Component(s) Affected
Checklist
pre-commit run --all-fileslocally and it passespytestor as relevant) and they passTesting
[How to test these changes]
Local Verification Evidence
Additional Notes
[Optional: any other context]
Summary by CodeRabbit
Documentation
Chores