Skip to content

Comments

fix(): further validate 404 reports#7543

Open
mhartington wants to merge 1 commit intomainfrom
404-reports
Open

fix(): further validate 404 reports#7543
mhartington wants to merge 1 commit intomainfrom
404-reports

Conversation

@mhartington
Copy link
Member

@mhartington mhartington commented Feb 21, 2026

Summary by CodeRabbit

  • Chores
    • Updated redirect configuration formatting for consistency and maintainability
    • Removed a redirect entry for an image resource path

@vercel
Copy link

vercel bot commented Feb 21, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
blog Ready Ready Preview, Comment Feb 21, 2026 6:35pm
docs Ready Ready Preview, Comment Feb 21, 2026 6:35pm

Request Review

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Feb 21, 2026

No actionable comments were generated in the recent review. 🎉


Walkthrough

The vercel.json configuration file undergoes structural reformatting of redirect entries from compact single-line format to expanded multi-line format with explicit fields. One redirect entry for a Prisma/Bun guide image is removed. Most remaining changes are stylistic formatting updates without altering routing behavior.

Changes

Cohort / File(s) Summary
Vercel Configuration Reformatting
apps/docs/vercel.json
Restructures redirect entries from single-line objects to multi-line format with explicit source, destination, and permanent fields. Removes one redirect mapping for /docs/img/guides/prisma-bun-cover-image.png. Adds expanded redirect blocks mirroring existing entries. Primary changes are formatting and structure; functional routing behavior unchanged except for the single removed redirect.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 inconclusive)

Check name Status Explanation Resolution
Title check ❓ Inconclusive The PR title 'fix(): further validate 404 reports' is vague and doesn't clearly describe the actual changes made to vercel.json redirect configurations. Consider a more specific title like 'fix(docs): restructure vercel.json redirect entries' or 'fix(docs): validate and deduplicate 404 redirects' to better reflect the JSON reformatting and redirect consolidation work.
✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

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


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.

@github-actions
Copy link
Contributor

🍈 Lychee Link Check Report

3666 links: ✅ 2991 OK | 🚫 9 errors | 🔀 0 redirects | 👻 664 excluded

❌ Errors

./apps/docs/content/docs.v6/(index)/prisma-orm/add-to-existing-project/mongodb.mdx

./apps/docs/content/docs.v6/(index)/prisma-orm/quickstart/mongodb.mdx

./apps/docs/content/docs.v6/guides/vercel-app-deployment.mdx

./apps/docs/content/docs.v6/orm/more/dev-environment/editor-setup.mdx

  • [503] https://fig.io/ | Rejected status code (this depends on your "accept" configuration): Service Unavailable

./apps/docs/content/docs.v6/orm/prisma-client/setup-and-configuration/databases-connections/index.mdx

./apps/docs/content/docs/(index)/prisma-orm/add-to-existing-project/mongodb.mdx

./apps/docs/content/docs/(index)/prisma-orm/quickstart/mongodb.mdx

./apps/docs/content/docs/guides/integrations/vercel-deployment.mdx

./apps/docs/content/docs/orm/more/dev-environment/editor-setup.mdx

  • [503] https://fig.io/ | Rejected status code (this depends on your "accept" configuration): Service Unavailable

Full Statistics Table
Status Count
✅ Successful 2991
🔀 Redirected 0
👻 Excluded 664
🚫 Errors 9
⛔ Unsupported 2
⏳ Timeouts 0
❓ Unknown 0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant