Repository navigation
docs: fix dev tools preview image path after website migration - #353
Conversation
|
@alcpereira is attempting to deploy a commit to the Callstack Team on Vercel. A member of the Team first needs to authorize it. |
|
No actionable comments were generated in the recent review.
|
| Layer / File(s) | Summary |
|---|---|
Update preview image links README.md, packages/dev-tools/README.md |
Both README files now link to the preview image under website/public instead of website/src/public. |
Priority:
Estimated code review effort: 1 (Trivial) | ~3 minutes
Change: Bug fix
Possibly related PRs
- callstackincubator/ai#351: Moves the preview image to
website/public; this pull request updates the README links to match.
Suggested reviewers: grabbou
Merge Risk: 
Minimal · up to 20740
-
The documentation links are correct and introduce no merge-blocking risk.
Architecture risk: 
Low · up to 20740
-
The change affects 2 systems.
Changed systems:
packages/dev-tools,README.mdArchitecture concerns
No architecture-level concerns identified.Review details
-
Systems and components
- observed — packages/dev-tools (library) was modified; 1 changed file maps to changed impact.
- observed — README.md (service) was modified; 1 changed file maps to changed impact.
Before / after behavior
- observed — Modified behavior in README.md: The DevTools preview image path changed from
website/src/public/dev-tools-preview.pngtowebsite/public/dev-tools-preview.png. - observed — Modified behavior in packages/dev-tools/README.md: The preview image link now uses
website/publicinstead ofwebsite/src/public.
-
Pre-merge checks: 
5
-
Full details: Title check
-
Explanation
The title "docs: fix dev tools preview image path after website migration" matches the main change. The PR updates the dev-tools preview image links in the root README.md and packages/dev-tools/README.md so they point to website/public/dev-tools-preview.png instead of website/src/public/. The fix follows the Astro migration that moved the file. The title is concise, specific, and follows a conventional commit prefix.
Full details: Docstring Coverage
-
Explanation
No eligible functions found in the changed files. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Documentation-only edits and deletions within surviving functions are included; unchanged diff context and fully deleted functions are excluded. Analyzed 0 functions across 0 files. (2 skipped: 2 unsupported.)
-
Finishing Touches: 💡 1
-
🛠️ Fix failing CI checks: 💡
-
- Commit to this branch
- Create a new PR
🧪 Generate unit tests (beta)
-
- Create a new PR
-
- Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts
Comment @coderabbitai help to get the list of available commands.
The Astro migration (#351) moved
dev-tools-preview.pngfromwebsite/src/public/towebsite/public/, leaving the image broken in the root README and thedev-toolspackage README. This points both at the new location.Summary by CodeRabbit