-
-
Notifications
You must be signed in to change notification settings - Fork 2k
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
Fix ANY
method not appearing in the UrlDispatcher
routes
#9987
Conversation
UrlDispatcher
routesANY
method not appearing in the UrlDispatcher
routes
CodSpeed Performance ReportMerging #9987 will not alter performanceComparing Summary
|
Codecov ReportAll modified and coverable lines are covered by tests ✅
✅ All tests successful. No failed tests found. Additional details and impacted files@@ Coverage Diff @@
## master #9987 +/- ##
=======================================
Coverage 98.73% 98.73%
=======================================
Files 121 121
Lines 36728 36732 +4
Branches 4384 4384
=======================================
+ Hits 36262 36266 +4
Misses 314 314
Partials 152 152
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
Backport to 3.11: 💔 cherry-picking failed — conflicts found❌ Failed to cleanly apply 7199e09 on top of patchback/backports/3.11/7199e09eceb279a90a1921f9dd784ad7a1d1b805/pr-9987 Backporting merged PR #9987 into master
🤖 @patchback |
Backport to 3.12: 💔 cherry-picking failed — conflicts found❌ Failed to cleanly apply 7199e09 on top of patchback/backports/3.12/7199e09eceb279a90a1921f9dd784ad7a1d1b805/pr-9987 Backporting merged PR #9987 into master
🤖 @patchback |
(cherry picked from commit 7199e09)
(cherry picked from commit 7199e09)
… the ``UrlDispatcher`` routes (#9988)
… the ``UrlDispatcher`` routes (#9989)
Regressed in #9899
The pseudo
ANY
method should still get added toself._routes
andself._allowed_methods
because external callers expect to inspect it.Discovered in aio-libs/aiohttp-apischema#44