Hi All, I could see a difference between gradle-de...

# detekt

[Venkat , Bangalore , India]

Hi All, I could see a difference between gradle-detekt-plugin (1.9.1) and detekt-sonar-plugin(2.0.0) reported issues... with same config yml file.. also anyone please tell me when can we expect detekt-sonar plugin will be updated based on 1.9.1?

[schalkms]

2.1.0 uses detekt 1.9.1
https://github.com/detekt/sonar-kotlin/commit/23d2c2d245c611c9f786e7a8bfdf3225f5451c23

[Venkat , Bangalore , India]

thanks for the update @schalkms 👍

[Artur Bosch]

I will request the addition to the marketplace tomorrow, expect it soon.

[Venkat , Bangalore , India]

cool. thanks for the update @Artur Bosch

[Venkat , Bangalore , India]

@Artur Bosch is it possible to add detekt config file in sonarqube itself . that way, we wanted to ensure that the team does not change yml file by adding exclusions just to fake results 🙂.

[Artur Bosch]

Yes, in the configuration of a project (only Admin) you can specify the config and baseline files. Best.

[Venkat , Bangalore , India]

@Artur Bosch thanks for your reply. I think as an admin i can only specify the Path of the config/baseline files.. correct? not the content to keep in server side

[Artur Bosch]

Yes, that's right. That's the easiest way to stay in sync with the project config isnt't it?

[Venkat , Bangalore , India]

ok, yep thats true .. was actually looking is there a to paste that config file in server side itself

[Venkat , Bangalore , India]

as long as developers are passionate enough to follow on local machines,, its good..

[Venkat , Bangalore , India]

some times, yeah ppl skip it

[Artur Bosch]

But pasting the config content for sonarqube only users would be still quite interesting. Never thought of it actually.

[Artur Bosch]

Feel free to open a feature request on the sonar-detekt side

[Venkat , Bangalore , India]

yeah thanks

[Venkat , Bangalore , India]

Also @Artur Bosch is there a ticket (new Rule) in your roadmap to enforce D in SOLID..

[Venkat , Bangalore , India]

making sure that "the constructor params are interfaces/abstract classes" of high level class

[Artur Bosch]

No such rule yet. Maybe you want to tackle such a rule ?

[Venkat , Bangalore , India]

yeah.. we wanted to enforce to follow SOLID strictly .. m just exploring writing a detekt custom rule..

[Venkat , Bangalore , India]

thats fine @Artur Bosch m really in love with Detekt .. awesome plugin 👍

[schalkms]

Thanks for the kind words! 🙂
Regarding the SOLID principles I have written numerous posts.
https://github.com/detekt/detekt/issues/1381
https://github.com/detekt/detekt/issues/2437