BTB icon indicating copy to clipboard operation
BTB copied to clipboard

Integrate with Codacy

Open lauragustafson opened this issue 7 years ago • 4 comments

Integrate with Codacy for automatic analysis/commenting on PR's for code quality and style. Would reduce a lot of the work in analyzing PR's.

lauragustafson avatar May 10 '18 17:05 lauragustafson

@micahjsmith @csala any comments?

kveerama avatar May 16 '18 15:05 kveerama

is this a general workflow tool, or are you targeting it specifically for people who contribute new tuners?

micahjsmith avatar May 16 '18 15:05 micahjsmith

It sounds interesting as an additional quality metric, and I would say that at least it's worth a try. However, I wouldn't expect it to be able to really replace the PR reviews.

Another option would be to give something like pylint a try, but it can become a bit annoying if we don't tweak the configuration to fit our our coding style preferences (maybe this also stands for codacy!)

csala avatar May 16 '18 18:05 csala

@micahjsmith It would be a general workflow tool, but we are using it to specifically target the contributions.

@csala It wouldn't completely replace the PR reviews but would help lessen the back and forth in contribution PR's. Essentially, we could wait to review a contribution-related PR (or really any) until after codacy accepts the code. At this point the code should be in pretty good shape, so reviewing a PR would take much less work. Also the default codacy settings do require a bit of tweaking in the configuration for some of our preferences (eg turning off some of the security-related checks). I integrated it into my fork of BTB and there weren't that many settings that needed tweaking and I found the comments/reviews pretty useful.

lauragustafson avatar May 16 '18 19:05 lauragustafson