Skip to content

Conversation

@alexdln
Copy link
Member

@alexdln alexdln commented Feb 11, 2026

Now we can go to https://chat.npmx.dev/package/nuxt, for example

Updated global redirects so they work for all paths within a subdomain

@vercel
Copy link

vercel bot commented Feb 11, 2026

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

Project Deployment Actions Updated (UTC)
npmx.dev Ready Ready Preview, Comment Feb 11, 2026 10:33pm
2 Skipped Deployments
Project Deployment Actions Updated (UTC)
docs.npmx.dev Ignored Ignored Preview Feb 11, 2026 10:33pm
npmx-lunaria Ignored Ignored Feb 11, 2026 10:33pm

Request Review

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Feb 11, 2026

📝 Walkthrough

Walkthrough

The vercel.json configuration file has been updated to modify all redirect source patterns. Each entry's source pattern has been changed from a single forward slash (/) to a catch-all pattern (/(.*)). This modification enables the redirects to match all paths within the specified routes whilst maintaining existing host-based gating logic and destination URLs. No changes have been made to other redirect properties or control flow mechanisms.

Suggested reviewers

  • danielroe
🚥 Pre-merge checks | ✅ 1
✅ Passed checks (1 passed)
Check name Status Explanation
Description check ✅ Passed The pull request description clearly relates to the changeset, explaining that redirect patterns were updated to work for all paths within subdomains, matching the vercel.json modifications.

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

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

No actionable comments were generated in the recent review. 🎉


Comment @coderabbitai help to get the list of available commands and usage tips.

@codecov
Copy link

codecov bot commented Feb 11, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ All tests successful. No failed tests found.

📢 Thoughts on this report? Let us know!

Copy link
Contributor

@serhalp serhalp left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Seems reasonable to me!

@serhalp serhalp added this pull request to the merge queue Feb 12, 2026
Merged via the queue into npmx-dev:main with commit e38b8ba Feb 12, 2026
17 checks passed
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.

2 participants