Repository navigation
Conversation
#9179 moved CTASection under Footer/components, so vale stopped on a missing argument and the Vale prose lint check fails on every PR.
|
No actionable comments were generated in the recent review.
|
| Layer / File(s) | Summary |
|---|---|
Update Vale target apps/marketing/package.json |
The lint:prose script points to the CTA section under components/Footer/components/CTASection. |
Estimated code review effort: 1 (Trivial) | ~3 minutes
Change: Bug fix
Suggested reviewers: saddlepaddle
Merge Risk: 
Minimal · up to 495f6
-
The focused path update is ready to merge.
Pre-merge checks: 
5
-
Full details: Description check
-
Explanation
The description explains the problem, identifies the moved CTASection path, and reports that bun run lint:prose passes with 0 errors. It does not include the template headings or checklist, but the core required information is present.
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 with cyclomatic complexity of at least 2. 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. (1 skipped: 1 unsupported.)
-
Finishing Touches
-
🧪 Generate unit tests (beta)
-
- Commit to this branch
- Create a new PR
-
- Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts
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 @coderabbitai help to get the list of available commands.
[Low impact] This path-only fix appears safe to merge. SummaryUpdates
Reviews (1) · Last reviewed commit: "fix(marketing): prose lint reads CTASect..." · Reviewed by Greptile |
🚀 Preview Deployment🔗 Preview Links
Preview updates automatically with new commits |
#9179 moved
CTASectiontoFooter/components/CTASection, butlint:proseinapps/marketing/package.jsonstill passes the old path. Vale stops with "argument ... does not exist", so the Vale prose lint check fails on every PR (e.g. #9184).One-line path fix.
bun run lint:prosepasses locally (0 errors).Summary by cubic
Fixes the marketing
lint:prosescript so the Vale prose lint check no longer fails on every PR.CTASectionmoved underFooter/components, but the script still passed its old path, which made Vale error out on a missing argument.Written for commit 495f640. Summary will update on new commits.
Summary by CodeRabbit