jdaugherty commented on code in PR #15826:
URL: https://github.com/apache/grails-core/pull/15826#discussion_r3525153863
##########
.github/workflows/gradle.yml:
##########
@@ -566,7 +566,7 @@ jobs:
MAVEN_PUBLISH_USERNAME: ${{ secrets.NEXUS_USER }}
MAVEN_PUBLISH_PASSWORD: ${{ secrets.NEXUS_PW }}
with:
- timeout_seconds: 1200
+ timeout_seconds: 1800 # headroom for slow Nexus uploads so a single
attempt is not killed mid-publish (see `publish` job)
Review Comment:
@jamesfredley why are we even defining timeout_seconds? I get that we used
to do this b/c of RAO misperforming, but shouldn't we just remove teh
timeout_seconds?
Also, we should merge changes without reviews ...
--
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]
