The GitHub Actions job "CI" on 
grails-core.git/feature/modernize-grails-async-8.0.x has succeeded.
Run started by GitHub user codeconsole (triggered by codeconsole).

Head commit for run:
8dd2b0a1dd3db9406fa1caa14df591a74df7e9ce / Scott Murphy Heiberg 
<[email protected]>
Merge upstream/8.0.x into feature/modernize-grails-async-8.0.x

One conflict, in AsyncActionResultTransformer: upstream renamed the
GrailsWebRequest accessor from getCurrentRequest() to getRequest() on the
same line this branch rewrote when it moved the async action result onto
Spring's DeferredResult. Resolved by keeping this branch's
DeferredResult/WebAsyncManager body and the `instanceof Promise promise`
pattern variable, with upstream's getRequest() rename applied.

Upstream also removed the multipart substitution that used to make
getCurrentRequest() and getRequest() differ, so the two accessors are now
the same object and this branch's remaining currentRequest call sites in
WebPromises and GrailsWebRequestTaskDecorator are unaffected.

The net-zero UrlMappingsHandlerMapping change on this branch left
upstream's rework of that class (Spring-assembled handler chains, the
hidden HTTP method rule) intact; its MATCHED_REQUEST short-circuit, which
this branch relies on to reuse the matched mapping across an async
dispatch, is unchanged.

Report URL: https://github.com/apache/grails-core/actions/runs/34713253822

With regards,
GitHub Actions via GitBox

Reply via email to