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
routes
macro allowing multiple paths per handler
#2718
Conversation
8a88765
to
f73c540
Compare
routes
macroroutes
macro allowing multiple paths per handler
This is now working and documented (already using it my self in https://github.com/ModProg/paste). Would like some feedback on the code to get this in a merge able state ASAP, so I don't need to run on a git version forever. |
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.
Nice!
Co-authored-by: Igor Aleksanov <popzxc@yandex.ru>
Is there anything left for me to do to make this PR mergeable? |
I haven't reviewed yet but I'm targetting this for v4.2, not the upcoming v4.1 so it will be merged after that release. |
this is dead code from back when .to and .to_async were different ways to add a service
CI issue was just missing a feature flag on Thanks for the contribution. |
Thanks, really looking forward to 4.2 now :) |
PR Type
Feature
PR Checklist
Overview
Adds support for
routes
macro, allowing multiple paths and methods to be specified.Closes #2713