jamesfredley commented on code in PR #15602:
URL: https://github.com/apache/grails-core/pull/15602#discussion_r3142024006
##########
.github/workflows/gradle.yml:
##########
@@ -454,6 +448,49 @@ jobs:
with:
name: grails-forge-artifacts.txt
path: grails-forge/build/grails-forge-artifacts.txt
+ verifyWrapper:
+ name: "Verify grails-wrapper"
+ # The wrapper smoke test downloads org.apache.grails:grails-cli from
Apache Nexus.
Review Comment:
Only the timing of when verification runs changes - moving it after
publishForge so the artifact grailsw needs (grails-cli) actually exists.
Without this, verification was relying on a stale snapshot from a prior CI run,
which is exactly why 7.1.x broke.
--
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]
