smengcl commented on issue #90: YETUS-941. Wrong URL in the checkstyle report. URL: https://github.com/apache/yetus/pull/90#issuecomment-588172690 Thanks [~aajisaka] for looking into this. Actually I also spent some time on this and figured that the culprit should be the `logfile` passed to `module_status` is the full path (should probably just be file name). I was still trying to figure out how to test the change. I'm glad you get the patch ready ahead of me. :D A question though, how do you usually test such change in Yetus? I looked at README and https://yetus.apache.org/contribute/. But there's no instruction on how to run Yetus locally to build and test hadoop trunk. e.g. Which set of commands should I use? Thanks! -Siyao
---------------------------------------------------------------- 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
