JiriOndrusek commented on PR #57: URL: https://github.com/apache/camel-upgrade-recipes/pull/57#issuecomment-3232202239
> The maven-surefire-plugin will be executed always, maybe it would be better to condition the execution? and update the deploy section in the README to enable the execution of the plugin? wdyt? I'd keep the execution of the 'latest.yaml tests' as is. here is why: - the second execution is much faster, because all the dependencies are already downloaded and loaded - in case we add any condition to run the tests for the second time (for latest.yaml), when do we expect to no run them? (CI needs to run them, developers would need to run them) - therefore I don't see any reason to skip them WDYT? -- 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]
