AEM-Rules-for-SonarQube
AEM-Rules-for-SonarQube copied to clipboard
wttech
SonarQube plugin with set of rules detecting possible bugs and bad smells specific for AEM development.
I think that the current implementation displays quite a lot of false positives, which diminish the value of this rule right now. Look at this example (https://github.com/Adobe-Consulting-Services/acs-aem-commons/blob/4b3da3f5fdf0b59049b966bea9306d24c87d0293/bundle/src/main/java/com/adobe/acs/commons/audit_log_search/impl/AuditLogSearchServlet.java#L64) ``` protected final...
I want to propose a new rule, which reports any usage of API call ```SlingSettingsService.getRunModes()```. In AEM projects, every dependency to a specific environment *must* be solved by creating a...
As per https://sling.apache.org/documentation/bundles/models.html#hints, the annotations `@Via`, `@Filter`, `@Named`, `@Optional`, `@Required`, `@Source` and `@Inject` have been replaced. Consider introducing a new rule to highlight their use and offer more modern suggestions.
Some of rules are not AEM-specific or should retire from various reasons. Prepare list and mark them as deprecated.
While going over the Wiki and the markdown files, I ran into a few files that look like they're supposed to show examples and counterexamples of valid/invalid code. Examples: -...
As the team was working on the last release, there was a bit of a kerfuffle around transitive dependency versions, which consumed some of our time. Consider using the [Maven...
`data-sly-test` had been known as the only way to assign a value to an identifier for a long time until `data-sly-set` was introduced for that purpose. Valid use of `data-sly-test`:...
1. Review current constants and validate whether current list holds up. 2. Find out whether there are significant differences between versions of AEM and constants. 3. Based on findings, prepare...
Prepare clear & concise contribution guidelines along with the vision for this project.
https://sonarcloud.io/dashboard?id=AEM-Rules-for-SonarQube