Jave-Chen opened a new pull request #2148: fix wrong sonar analysis wrong coverage with checkout V2. URL: https://github.com/apache/incubator-dolphinscheduler/pull/2148 ## What is the purpose of the pull request with checkout V2, sonar analysis get warning like: ``` ... [WARNING] Missing blame information for the following files: [WARNING] * dolphinscheduler-remote/src/main/java/org/apache/dolphinscheduler/remote/command/ExecuteTaskResponseCommand.java [WARNING] * dolphinscheduler-remote/src/main/java/org/apache/dolphinscheduler/remote/command/CommandType.java [WARNING] * dolphinscheduler-common/src/main/java/org/apache/dolphinscheduler/common/utils/DateUtils.java [WARNING] * dolphinscheduler-remote/src/main/java/org/apache/dolphinscheduler/remote/command/ExecuteTaskRequestCommand.java [WARNING] This may lead to missing/broken features in SonarCloud ... ``` and then sonar analysis get the wrong coverage for PR ## Brief change log 1. add both push and pull request for ui_ct workflow. 2. upload coverage report just run on pull request event. 3. after test, run git command to convert a shallow repository to a complete one. ``` git fetch --unshallow git config remote.origin.fetch "+refs/heads/*:refs/remotes/origin/*" git fetch origin ``` ## Verify this pull request test on my repo https://github.com/Jave-Chen/incubator-dolphinscheduler/pull/9/checks?check_run_id=500815729 there isn't any blame warning.
---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: [email protected] With regards, Apache Git Services
