codeconsole commented on PR #16183: URL: https://github.com/apache/grails-core/pull/16183#issuecomment-5469962068
Consolidated into #16149, which now covers the request path as one reviewable change — per-request work, hidden HTTP method handling and multipart together. This branch was merged in with its history intact; no commits were squashed or dropped, and `git merge-base --is-ancestor` confirms every commit here is contained in #16149. Closing in favour of that. The reason is review flow rather than anything wrong here: these branches collided in `GrailsDispatcherServlet.checkMultipart` and `UrlMappingsHandlerMapping`, so reviewing them separately meant tracking a merge order, which is not a reasonable thing to ask of a reviewer. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
