Skip to content

docs(community): add clear router#1303

Merged
pi0 merged 5 commits intoh3js:mainfrom
3m1n3nc3:patch-2
Feb 16, 2026
Merged

docs(community): add clear router#1303
pi0 merged 5 commits intoh3js:mainfrom
3m1n3nc3:patch-2

Conversation

@3m1n3nc3
Copy link
Contributor

@3m1n3nc3 3m1n3nc3 commented Feb 16, 2026

resolves #1302

Summary by CodeRabbit

  • Documentation

    • Added Clear Router to the Community section with feature overview including Laravel-style routing, middleware, controller bindings, TypeScript support, and GitHub repository link.
  • Style

    • Minor code formatting adjustment.

@3m1n3nc3 3m1n3nc3 requested a review from pi0 as a code owner February 16, 2026 18:13
@coderabbitai
Copy link

coderabbitai bot commented Feb 16, 2026

📝 Walkthrough

Walkthrough

This pull request adds documentation for Clear Router, a Laravel-style routing system for H3 and Express.js, to the community section while applying a minor formatting adjustment to a function declaration in the handler code.

Changes

Cohort / File(s) Summary
Community Documentation
docs/2.utils/99.community.md
Adds new Clear Router entry with description of features (Laravel-style routing, middleware, controller bindings, TypeScript support) and link to GitHub project.
Code Formatting
src/handler.ts
Removes space before parenthesis in resolveLazyHandler function declaration within defineLazyEventHandler.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

Poem

🐰 A router so clear, now the world gets to know,
With Laravel-style grace and TypeScript aglow,
And formatting tidied with spacing just right,
Our community grows, shining ever more bright! ✨

🚥 Pre-merge checks | ✅ 4 | ❌ 2

❌ Failed checks (2 warnings)

Check name Status Explanation Resolution
Out of Scope Changes check ⚠️ Warning The PR includes an unrelated minor formatting change in src/handler.ts (function spacing), which is outside the scope of adding Clear Router documentation. Remove the unrelated formatting change in src/handler.ts or create a separate PR for that minor adjustment to keep changes focused.
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (4 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Linked Issues check ✅ Passed The PR adds Clear Router to the Community section with required details (routing style, middleware, controller bindings, TypeScript support), fulfilling issue #1302 requirements.
Merge Conflict Detection ✅ Passed ✅ No merge conflicts detected when merging into main
Title check ✅ Passed The title 'docs(community): add clear router' is specific and directly summarizes the main change—adding Clear Router to the community documentation.

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

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

Tip

Issue Planner is now in beta. Read the docs and try it out! Share your feedback on Discord.


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.

@pkg-pr-new
Copy link

pkg-pr-new bot commented Feb 16, 2026

Open in StackBlitz

npm i https://pkg.pr.new/h3js/h3@1303

commit: 12c03e5

@pi0 pi0 changed the title Add Clear Router section to community.md docs(community): add clear router Feb 16, 2026
Copy link
Member

@pi0 pi0 left a comment

Choose a reason for hiding this comment

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

thnx!

@pi0 pi0 merged commit 1f28f59 into h3js:main Feb 16, 2026
3 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.

Add Clear Router to Community Section

2 participants