You are viewing a plain text version of this content. The canonical link for it is here.

Posted to gitbox@yetus.apache.org by GitBox <gi...@apache.org> on 2020/02/19 11:25:19 UTC

[GitHub] [yetus] smengcl commented on issue #90: YETUS-941. Wrong URL in the checkstyle report.

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:
users@infra.apache.org


With regards,
Apache Git Services