You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@grails.apache.org by "jamesfredley (via GitHub)" <gi...@apache.org> on 2026/09/16 22:53:43 UTC

[PR] Fix Groovy 5 duplicate resp in grails-multi-datasource v8 test snippet [grails-static-website]

jamesfredley opened a new pull request, #552:
URL: https://github.com/apache/grails-static-website/pull/552

   ## Summary
   Companion `complete/` integration spec failed to compile on Grails 8 / Groovy 5 because `resp` was declared in the save loops and again later in the same method.
   
   Rename those locals to `saveResp` so the published snippet matches grails-guides/grails-multi-datasource#3.
   
   Companion: https://github.com/grails-guides/grails-multi-datasource/pull/3
   
   ## Test plan
   - [ ] Website CI (Java CI / build) is green
   - [ ] Snippet matches the companion `MultipleDataSourceSpec` fix


-- 
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: commits-unsubscribe@grails.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] Fix Groovy 5 duplicate resp in grails-multi-datasource v8 test snippet [grails-static-website]

Posted by "jamesfredley (via GitHub)" <gi...@apache.org>.
jamesfredley merged PR #552:
URL: https://github.com/apache/grails-static-website/pull/552


-- 
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: commits-unsubscribe@grails.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org