The GitHub Actions job "Code Style" on grails-core.git/7.0.x-remove-EnableWebMvc has failed. Run started by GitHub user codeconsole (triggered by codeconsole).
Head commit for run: 5c4974389bf7e6c25bd7fc2b63bf6a93960b1459 / Scott Murphy Heiberg <[email protected]> Address review: docs, config metadata, and functional coverage for form parsing Responds to review feedback on the @EnableWebMvc removal: - Docs (upgrading80x.adoc): the upgrade notes were out of date. Boot's static- resource handler and welcome page are now removed by default (new section 31.3), not "active"; renumber the remaining Boot-MVC notes to 31.4; add 31.5 for locale resolution. Clarify that the form-content filter only affects application/x-www-form-urlencoded bodies (also in the auto-config javadoc). - Docs (changingLocales.adoc): spell out precedence — with session/cookie the Accept-Language header sets the initial locale and ?lang= overrides and persists it; with acceptHeader/fixed the resolver is read-only and ?lang= is ignored. - Config metadata: add grails.web.removeDefaultViewResolverBean, grails.web.removeWelcomePageMapping and grails.i18n.localeResolver (with value hints) to additional-spring-configuration-metadata.json. - Tests: since GrailsParameterMap no longer parses PUT/PATCH bodies itself, add a functional test in app1 (a default, non-@EnableWebMvc app) asserting form-encoded PUT/PATCH/DELETE/POST bodies are parsed into params via Boot's OrderedFormContentFilter. - Remove the enable-mvc-check README (it only existed to explain the test). Report URL: https://github.com/apache/grails-core/actions/runs/29031281680 With regards, GitHub Actions via GitBox
