Skip to content

Commit

Permalink
updated evaluate rules
Browse files Browse the repository at this point in the history
added example to rule
  • Loading branch information
jborgers committed May 2, 2024
1 parent 44375f0 commit 475799a
Show file tree
Hide file tree
Showing 3 changed files with 145 additions and 5 deletions.
2 changes: 1 addition & 1 deletion docs/EvaluateRulesForKotlin.md
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@ Evaluation of Java PMD rules for use as Kotlin rules
| # | Who does | Rule name | useful | complexity | used by sponsors | importance | already available | note / to investigate |
|-----|---------------|------------------------------------------|--------|------------|------------------|-------------|-------------------|-------------------------------------------------------------------------------------------------------------------------|
| 1. | | AvoidCDIReferenceLeak | Yes | Medium | No | Low | Not found | Kotlin mostly not used with Java/JakartaEE |
| 2. | | AvoidConstantsInInterface | ? | Low? | Yes | Low | Not found | To investiate |
| 2. | | AvoidConstantsInInterface | ? | Low? | Yes | Low | Not found | To investigate |
| 3. | PP-1-M | AvoidDecimalAndChoiceFormatAsField | Yes? | Low | Yes | High | Not found | NumberFormat/DateFormat not included? |
| 4. | | AvoidDuplicateAssignmentsInCases | Yes | Medium | Yes | Low/Medium | Partly found | Detekt:DuplicateCaseInWhenExpression has overlap but is not the same. Add example, doc, Questionable if occuring often. |
| 5. | PP-6---WIP | AvoidImplicitlyRecompilingRegex | Yes | High | Yes | High | Not found | Kotlin has own String/regex, also occurs here? support both? |
Expand Down

0 comments on commit 475799a

Please sign in to comment.