Ecosyste.ms: Awesome

An open API service indexing awesome lists of open source software.

Awesome Lists | Featured Topics | Projects

https://github.com/mkohm/detekt-hint

Detection of design principle violations in Kotlin as a plugin to detekt.
https://github.com/mkohm/detekt-hint

anti-patterns code-quality code-smells danger design-principles detections detekt detekt-hint detekt-plugin kotlin programming-principles software-architecture static-code-analysis

Last synced: 14 days ago
JSON representation

Detection of design principle violations in Kotlin as a plugin to detekt.

Awesome Lists containing this project

README 

        



# detekt-hint (Attention: Looking for developers)

Detection of design principle violations in Kotlin added as comments on pull requests.



## [Getting started](#getting-started) | :checkered_flag: [Roadmap](#upcoming-features) | :thought_balloon: [Ask a question](https://github.com/Mkohm/detekt-hint/issues/new) | :book: [Documentation](https://mkohm.github.io/detekt-hint/docs/)



[![Maintainability](https://api.codeclimate.com/v1/badges/307995daba5f21506f4d/maintainability)](https://codeclimate.com/github/Mkohm/detekt-hint/maintainability) [![codecov](https://codecov.io/gh/Mkohm/detekt-hint/branch/master/graph/badge.svg)](https://codecov.io/gh/Mkohm/detekt-hint) [![Build Status](https://travis-ci.com/Mkohm/detekt-hint.svg?branch=master)](https://travis-ci.com/Mkohm/detekt-hint) [ ![Download](https://api.bintray.com/packages/bintray/jcenter/io.github.mkohm%3Adetekt-hint/images/download.svg) ](https://bintray.com/bintray/jcenter/io.github.mkohm%3Adetekt-hint/_latestVersion)

[![Kotlinlang slack](https://img.shields.io/static/v1?label=kotlinlang&message=detekt-hint&color=brightgreen&logo=slack&style=flat-square)](https://app.slack.com/client/T09229ZC6/C012ZSM6L1J)

[![Awesome Kotlin Badge](https://kotlin.link/awesome-kotlin.svg)](https://github.com/KotlinBy/awesome-kotlin)



![ucih](images/demo.png)






detekt-hint is a plugin for [detekt](https://github.com/arturbosch/detekt) that includes detection of violation of programming principles. Since such violations are hard to detect with low false-positive rates, detekt-hint will provide hints during QA, minimizing noise during development. The idea is that a higher false-positive rate can be accepted if the detection could be of high value, and is easy to ignore. Detections on the architectural level of code is therefore most likely to provide value.



Through integration with [Danger](https://github.com/danger/danger) comments are added to the PR. Getting feedback directly on the PR makes it easy to ignore possible false-positives. Comments also include context and tips, making it easier for the developer to make the correct decisions. 



Contributions are very much welcome and if you like the project - help me out with a :star:. Especially help in which rules to implement, how to implement them and how to reduce the false-positives.



## Currently supported detections

:white_check_mark: Use composition instead of inheritance - Will help developer ensure Liskov Substitution Principle is not violated. Will not report if you inherit from third-party libraries.



:white_check_mark: Lack Of Cohesion of Methods - Notifies you if the LCOM value is too high.



:white_check_mark: Interface Segregation Principles - Notifies you if you implement methods that the class does not need.



:white_check_mark: Open-Closed Principle - This rule reports use of switching on enums and classes, which may be a sign of violation the open closed principle.



### Interface Segregation Principle

![lcom](images/comment_isp.png)



### Single Responsibility Principle

![lcom](images/comment_lackOfCohesion.png)



### Open-Closed Principle

![lcom](images/comment_ocp2.png)



## Using detekt-hint

If you just want to analyze some code without bothering with the Danger integration (which really defeats the purpose of the tool) head to the [command line section](#With-the-command-line).



### With Github Actions

1. Configure a detekt-hint-config.yml to include detekt-hint rules and put it in a folder called "config" in your root project folder. 



config/detekt-hint-config.yml

```yml

detekt-hint:

  UseCompositionInsteadOfInheritance:

    active: true

    yourUniquePackageName: "io.github.mkohm"

  LackOfCohesionMethods:

    active: true

    threshold: "0.8"

  InterfaceSegregationPrinciple:

      active: true

  OpenClosedPrinciple:

      active: true

```



Make sure you enter your unique package name in the configuration for the UseCompositionInsteadOfInheritance rule.



2. Create a github action using the detekt-hint docker action. 



.github/workflows/detekt-hint.yml

```yml

name: detekt hint



on:

  pull_request:

    branches:

      - '*'



jobs:

  gradle:

    strategy:

      matrix:

        os: [ubuntu-latest]

        jdk: [11]

    runs-on: ${{ matrix.os }}

    if: ${{ !contains(github.event.head_commit.message, 'detekt hint skip') }}

    env:

      JDK_VERSION:  ${{ matrix.jdk }}

    steps:

      - name: Checkout Repo

        uses: actions/checkout@v2

        

      - name: Run detekt hint

        uses: mkohm/[email protected]

        with:

          github-api-token: ${{ secrets.GITHUB_TOKEN }}

```



3. Create a PR and see detekt-hint run as a separate action.



Having trouble? Please [create an issue](https://github.com/Mkohm/detekt-hint/issues/new) or contact me on the kotlinlang Slack (username: mkohm), and i will help you out.



### With the command line

If you only want to do some analysis on your code without the power of Danger commenting on your PR you can use the tool from the command line. You must first clone detekt and detekt-hint repositories, and then build the required jars:

```

git clone https://github.com/Mkohm/detekt-hint && git clone https://github.com/arturbosch/detekt && cd detekt-hint && ./gradlew jar && cd ../detekt/ && ./gradlew build shadowJar && cd ..

```

Use the command line utility:

```bash

java -jar detekt/detekt-cli/build/libs/detekt-cli-[version]-all.jar --plugins detekt-hint/build/libs/detekt-hint-[version].jar --config detekt-hint/config/detekt.yml --classpath  --input 

```

For example:

```bash

java -jar detekt/detekt-cli/build/libs/detekt-cli-1.5.0-all.jar --plugins detekt-hint/build/libs/detekt-hint-0.0.2.jar --config detekt-hint/config/detekt.yml --classpath detekt-hint/ --input detekt-hint/

```

Remember to enter the [latest](https://mvnrepository.com/artifact/io.gitlab.arturbosch.detekt/detekt-cli) detekt-cli version, the [latest](https://mvnrepository.com/artifact/io.github.mkohm/detekt-hint) detekt-hint version and the path to your classpath and source code. Also, make sure that the detekt.yml you are using contains the unique package name in the configuration for the UseCompositionInsteadOfInheritance rule.