alexstyl / contactstore

A modern, strongly-typed contacts API for Android.
https://alexstyl.github.io/contactstore
Apache License 2.0
440 stars 15 forks source link

Create static analysis gradle task (detekt + lint) #28

Closed alexstyl closed 1 year ago

alexstyl commented 2 years ago

There are more people contributing to the project lately, and it would be nice to have less moderation required on each PR. This issue is about creating a gradle task to run static analysis in the project.

Requirements:

Keep in mind that there are going to be way more detekt and lint issues than the default thresholds. There is no need to fix anything in the code that the static analysis will pick up. Instead, increases the threshold of allowed issues to the number of issues detected.

There is no need for this work to take place in a single PR.

If you would like to pick up this task, reply to this issue, so that I can assign it to you.

Link to Detekt: https://detekt.github.io/detekt/ Example Project with static analysis set up: https://github.com/code-with-the-italians/bundel/

tatocaster commented 2 years ago

No need to increase the threshold, they should be strict to enforce good practices in future development Major lint issues can be fixed right away and others can be suppressed with the baseline and from time to time contributors can fix it from baseline - a good opportunity for future contributors and newcomers.

alexstyl commented 1 year ago

Closing this as it's been more than a year open