Are you going to share your code quality pattern settings?
It would be great to handle this like code formatting so we can also harmonize the code quality of modules in general. We could just export the settings as json and put it into the core repo with the formatter xml.
And if so, is there going to be a debate/vote on what patterns we want to adhere to?
@teleivo, Iβm still experimenting and exploring the rules. Once we settle on them I think itβs a good idea to export the rules and version in our repository so that they can be used to configure other projects. As Lluis noted our core code is in a pretty good shape already thanks to fixes we made when introducing Sonar and we should simply match codacy rules with those from Sonar.
I think codacy is a much better tool for our needs than Sonar. It enables pull request checking and is easier to setup than Sonar for individual projects.
For what I see, Codacy uses PMD (Sonar too). But yes the PR integration is pretty nice.
PMD has debatable rules, specially the ones about long methods, long classes. Rules that even the Java API violates
exactly what I mean modules just cannot use openmrs sonar at the moment, as far as I know. so this would be a great way to get them in line with the core as well