-
-
Notifications
You must be signed in to change notification settings - Fork 87
Update example URL port in testing/panel/README.md #168
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. Weβll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
WalkthroughA port number in an example URL is updated from 3001 to 3010 in the testing/panel README documentation. Changes
Estimated code review effortπ― 1 (Trivial) | β±οΈ ~1 minute No functional changes; straightforward documentation update. Poem
Pre-merge checks and finishing touchesβ Passed checks (3 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.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
This PR corrects the port number in the README documentation to match the actual configuration used by the development server. The README was incorrectly referencing port 3001, while the actual dev script in package.json uses port 3010.
- Updated localhost port reference from 3001 to 3010 in the usage instructions
π‘ Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 1
π Review details
Configuration used: defaults
Review profile: CHILL
Plan: Pro
π Files selected for processing (1)
testing/panel/README.md(1 hunks)
π§° Additional context used
π§ Learnings (1)
π Learning: 2025-12-13T17:09:09.794Z
Learnt from: CR
Repo: TanStack/ai PR: 0
File: CLAUDE.md:0-0
Timestamp: 2025-12-13T17:09:09.794Z
Learning: Applies to examples/** : Examples are not built by Nx and should be run independently from their directories with `pnpm dev` or `pnpm install && pnpm dev`
Applied to files:
testing/panel/README.md
πͺ markdownlint-cli2 (0.18.1)
testing/panel/README.md
21-21: Bare URL used
(MD034, no-bare-urls)
| ``` | ||
|
|
||
| Then open http://localhost:3001 | ||
| Then open http://localhost:3010 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
π§© Analysis chain
π Script executed:
# Search for port configuration in testing/panel directory
find testing/panel -type f \( -name "*.json" -o -name "*.js" -o -name "*.ts" -o -name "*.config.*" -o -name "vite.config.*" -o -name "next.config.*" \) 2>/dev/null | head -20Repository: TanStack/ai
Length of output: 820
π Script executed:
# Check package.json for dev script and port configuration
cat testing/panel/package.jsonRepository: TanStack/ai
Length of output: 1378
π Script executed:
# Check app.config.ts for any additional port configuration
cat testing/panel/app.config.tsRepository: TanStack/ai
Length of output: 318
Wrap the bare URL in angle brackets to comply with markdown linting standards.
The bare URL violates the MD034 (no-bare-urls) markdown linting rule. Without angle brackets, the URL isn't converted into a link in many markdown parsers. Wrap it in angle brackets:
-Then open http://localhost:3010
+Then open <http://localhost:3010>π Committable suggestion
βΌοΈ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
| Then open http://localhost:3010 | |
| Then open <http://localhost:3010> |
π§° Tools
πͺ markdownlint-cli2 (0.18.1)
21-21: Bare URL used
(MD034, no-bare-urls)
π€ Prompt for AI Agents
In testing/panel/README.md around line 21 the bare URL "http://localhost:3010"
violates MD034 (no-bare-urls); update the line to wrap the URL in angle brackets
so the markdown linter treats it as a link (replace the bare URL with
<http://localhost:3010>).
π― Changes
Update the example URL in
testing/panel/README.mdto use port 3010 instead of 3001.β Checklist
pnpm run test:pr.π Release Impact
π Details & Motivation
The documentation previously referenced port 3001, but the application default has changed to 3010. This update ensures new users can copy-paste the example command and successfully connect to the panel without connection errors.
Summary by CodeRabbit
βοΈ Tip: You can customize this high-level summary in your review settings.