jacopoc commented on PR #1033: URL: https://github.com/apache/ofbiz-framework/pull/1033#issuecomment-4188659225
@ribafish Now that I’ve merged your other pull request #1028, which made the `test` task cache-friendly, we could also merge this pull request. Currently, our CI [workflow](https://github.com/apache/ofbiz-framework/blob/trunk/.github/workflows/gradle.yaml) runs: `./gradlew check javadoc` However, our goal is to switch to the following as soon as we resolve some build issues: `./gradlew pullAllPluginsSource check javadoc loadAll testIntegration` My only concern is that merging this PR before we understand and fix the issues preventing us from running the new tasks might make it more difficult to debug why they fail on CI but succeed locally. Moreover, since `loadAll` is not currently cache-friendly, enabling it may again lead to a significant number of cache misses. I’ll try to investigate these issues, but if you have any advice, please let me know. Thank you in advance! -- 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]
