codeconsole commented on PR #16180: URL: https://github.com/apache/grails-core/pull/16180#issuecomment-6349366078
@jdaugherty Thanks for checking the native run. Both points from the review body are done: - **Description:** the double-injection paragraph is gone. I also scoped the Limitations section to starting the application, to match your guide comment, and retitled the PR to "Start Grails as a native image…". - **History:** the 11 commits that duplicated #16709 are gone. Every commit after them was replayed in order (`3dee35205b`), so the branch now has 35 commits of its own, and the fix appears in the history once, through 9.0.x. I kept the 9.0.x merges. None of them carries a conflict resolution (`git show --remerge-diff` is empty for all 23), so they only bring 9.0.x in. The new head has exactly the same files as the head before the rewrite. The three inline comments are answered in their threads, with the commits that address them. -- 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]
