SaptarshiSarkar12 commented on PR #13489: URL: https://github.com/apache/dubbo/pull/13489#issuecomment-1884436981
> 1. How about set `Check if code needs formatting` as the required step for all the steps in `Build and Test For PR`. This means we should move `Check for formatting` into `Build and Test For PR`. @AlbumenJ Yeah, that's a good idea. > 2. Why use `--log-file mvn.log` in `Check if code aligns with code style` @AlbumenJ I have tried using `gh run view --log` command to get the log, but it does not work until the run completes its execution. So, I have used that arg to dump the maven execution output to a log file from which I'll extract the error and show it in the Workflow summary. -- 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]
