jomarko commented on issue #1726: URL: https://github.com/apache/incubator-kie-issues/issues/1726#issuecomment-2567698119
Thank you @martinweiler for reporting this issue. I wonder what could be our strategy here. As searching for `System.out.println` in `src/test/**/*.bpmn2` or `src/test/**/*.bpmn` returns in total ~494 results. 1. **simply remove all `System.out.println(.*);` substrings with empty strings?** Is a script task with empty content valid for the engine? I think this could be safe approach if current scripts do not execute code like `setNewValueForVariable()` 2. **replace `System.out.println` with some `log.debug`**? or other low log level, that is not visible by default? 3. **remove just `System.out.println` that are part of more complex scripts?** I think this is scenario of linked discussion 4. Do we want to update only bpmn files or also the Test.java files? -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
