Local directory prefix is not correctly being removed. This is the link issue.
I'm not sure that displaying them in the Slack is the right thing to do. Perhaps a link on a status? A collapse/expand button would make them useful in Slack.
Checkstyle doesn't provide auto fixing, so we'd need to. This is out of scope for RI, as far as Checkstyle goes. We could use some of our own reviewers (like the "Phil" reviewers) if we want auto fix.
Comment by @johnsonr at 2018-03-07T18:10:21Z:
Significant interest in Checkstyle integration at QCon. Worth prioritizing
Issue moved from atomist/sdm#46
Created by @jessitron at 2018-02-21T19:17:29Z:
to get these to be pretty, they need:
Comment by @johnsonr at 2018-02-22T01:03:53Z:
Local directory prefix is not correctly being removed. This is the link issue.
I'm not sure that displaying them in the Slack is the right thing to do. Perhaps a link on a status? A collapse/expand button would make them useful in Slack.
Checkstyle doesn't provide auto fixing, so we'd need to. This is out of scope for RI, as far as Checkstyle goes. We could use some of our own reviewers (like the "Phil" reviewers) if we want auto fix.
Comment by @johnsonr at 2018-03-07T18:10:21Z:
Significant interest in Checkstyle integration at QCon. Worth prioritizing