-
Notifications
You must be signed in to change notification settings - Fork 702
RFC-7130: Route Layer #7130
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
RFC-7130: Route Layer #7130
Conversation
MrCroxx
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!This is a very good feature.
There’s one thing I’m not quite sure about: when specifying the purpose of a route in the fn route() method, is it better to use an Operator or an Accessor? Although the boundary between the two doesn’t seem very strict at the moment, using an Accessor might allow for more complex composition rules?
My current plan is to use
Ideally, the |
flaneur2020
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM! lookin really nice
koushiro
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. It's an interesting idea.
Which issue does this PR close?
None
Rationale for this change
Add
RouteLayerso that users can route requests based on path.What changes are included in this PR?
We gonna build something to allow users to:
Are there any user-facing changes?
AI Usage Statement
Parts of this PR were drafted with assistance from Codex (with
gpt-5.2) and fully reviewed and edited by me. I take full responsibility for all changes.