jamesfredley opened a new pull request, #16141:
URL: https://github.com/apache/grails-core/pull/16141
## Summary
Expand the Grails 8 upgrade guide with symptom-driven guidance for using
plugins built for Grails 7.
The new documentation:
- distinguishes application-side workarounds from failures that require a
plugin rebuild
- documents the compatibility failures reproduced in #16122 through #16128
- adds before-and-after examples for `Metadata`, Jackson 2 dataformats,
pre-Apache dependency coordinates, and Spring 7 `HttpHeaders`
- links the already-shipped legacy command, CLI companion, script packaging,
and plugin bean lifecycle behavior from #15934, #15948, #16011, and #16082
- corrects the existing guide's `Holders.grailsApplication` timing statement
- removes stale Spring Security and SiteMesh blocker guidance
## Compatibility cases covered
- removed `Metadata` dynamic and map-style access
- Groovy 4 generic trait field-helper bytecode
- Groovy 4 AST transformations targeting private trait methods
- Groovy 4 `@Immutable` bytecode referencing removed `ObjectUtil`
- incomplete Jackson 2 Smile, CBOR, or YAML runtime classpaths
- versionless dependencies under pre-Apache `org.grails` coordinates
- Spring 6 `HttpHeaders` bytecode running against Spring Framework 7
- Grails 7 command plugins and retimed `doWithSpring` callbacks
## Verification
- `./gradlew.bat :grails-doc:publishGuide -x :grails-doc:aggregateGroovydoc
--max-workers=13`
- Chromium QA of the generated `guide/upgrading.html`
- all six new subsections rendered
- code examples rendered
- all seven issue links and four PR links were present
- all four internal cross-references were clicked and reached visible
targets
- no browser console errors
- `git diff --check`
- mandatory Oracle review: GREEN
- mandatory Codex review: GREEN
- Checkstyle, CodeNarc, PMD, and SpotBugs aggregate reports: no violations
- aggregate unit tests: 12,316 passed, 0 failures
The mandatory aggregate integration run reached 3,617 tests and reported two
unrelated failures:
- `grails-test-examples-mail`: a Windows-only CRLF assertion expected
`Hello\nWorld!` but received `Hello\r\nWorld!`; it reproduced in an isolated
serial rerun
- `grails-test-examples-scaffolding`: a browser test timed out on the
welcome page during the full parallel run; the exact spec passed in an isolated
serial rerun
## Related issues
- #16122
- #16123
- #16124
- #16125
- #16126
- #16127
- #16128
--
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]
