Segun-Ogundipe commented on pull request #4299: URL: https://github.com/apache/incubator-dolphinscheduler/pull/4299#issuecomment-758637792
> > What part should I split into another PR. The test coverage needs to be increased to at least 33% to pass sonar check. Should I leave test coverage out of this PR? > > you can split it according the module, like dolphinscheduler-alert、dolphinscheduler-api and so on. just remember don't includes too many files in one PR. because review these files will be difficult > > please contact me ([[email protected]](mailto:[email protected])) when meeting any problem Thanks for the explanation. I'll get right on it ---------------------------------------------------------------- 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]
