search

wttech / AEM-Rules-for-SonarQube

SonarQube plugin with set of rules detecting possible bugs and bad smells specific for AEM development.
Apache License 2.0
112 stars 51 forks source link

Unable to get all the AEM rules after installing v1.2 to SonarQube #203

Closed singhs1704 closed 2 years ago

singhs1704 commented 3 years ago

Hi Team,

I have SonarQube Community Edition v8.2 installed on a Docker image and I have installed the latest AEM SQ Rules v1.2 plugin on it. After completing the installation and restarting the server I only get the HTL Quality Profile with 17 rules in it but I do not get any Java AEM rules or Quality profile installed into SonarQube. Could you please advise?

Thanks

singhs1704 commented 3 years ago

Hi Team,

has anyone faced this issue? Any pointers here would be very helpful.

Thanks

toniedzwiedz commented 3 years ago

Hi @singhs1704, sorry I missed your message earlier. I haven't seen this behaviour. Can you provide more detail as to what's going on? Are these rules not used by your analyss? Do you not see the rules at all when inspecting https://sonarqube.example.com/coding_rules ?

singhs1704 commented 3 years ago

Hi @toniedzwiedz, when I go into the quality profiles section I can see an additional Quality profile by the name AEM HTL profile but that only has 19 rules in it. I would have expected approximately 117 custom AEM rules instead. Please let me know if this is incorrect?

singhs1704 commented 3 years ago

Hi @toniedzwiedz Sorry for chasing again. Do you have any pointers for me on this issue?

TianZuo555 commented 3 years ago

same thing here...

update: I can see rules by searching AEM as tag not as language

toniedzwiedz commented 3 years ago

Hi, most rules are related to Java code (so they're not part of the HTL profile, which extends the HTML profile with HTL-specific rules) and they're tagged with AEM, as @TianZuo555 noted. This hasn't changed in any of the recent versions. @singhs1704 is it that the rules don't fire when expected or is it just about their potentially confusing location?

toniedzwiedz commented 2 years ago

Closing this one due to lack of activity. Please reopen if this is still an issue.