-
Notifications
You must be signed in to change notification settings - Fork 29
Conversation
with sample setup
Configure detekt using the gradle plugins repo
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for this @takecare. I left a minor question, LGTM otherwise.
|
||
<suppressions> | ||
|
||
<suppress checks=".*" files=".*\.kt" /> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
wouldn't be enough to have this setting in the buildscript using the checkstyle.exclude()
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
indeed. and it's another plugin functionality being showcased.
to showcase different ways of ignoring/excluding tests
and fix pmd rule set import
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice — I would've put a mention to the sample in the repos/documentation though (and, possibly, some mention in the sample's own readme of how the Novoda scaffolding works)
sample/app/build.gradle
Outdated
buildToolsVersion '27.0.3' | ||
|
||
defaultConfig { | ||
applicationId "com.novoda.staticanalysis.sample" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nit: single quotes here and for the versionName
sample/build.gradle
Outdated
@@ -0,0 +1,20 @@ | |||
buildscript { | |||
ext.kotlin_version = '1.2.21' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why not 1.2.30?
@mr-archano yup, i'll open another pr for the multi-module setup later |
Ok let's 🚢 then! Thanks @takecare :) |
Adds a sample app showcasing the usage of the plugin and some of its features. The inclusion of
detekt
might be particularly interesting since its setup is not straightforward.Should cover #95