Replace @active KDoc tag with @ActiveByDefault annotation

[marschwar]

Following up on #3562 this replaces the @active¸ kdoc tag with an @ActiveByDefault` annotation.

[codecov]

Codecov Report

Merging #3592 (a31528f) into main (92b6a01) will increase coverage by 0.01%.
The diff coverage is 82.85%.

❗ Current head a31528f differs from pull request most recent head adc79ae. Consider uploading reports for the commit adc79ae to get more accurate results

@@             Coverage Diff              @@
##               main    #3592      +/-   ##
============================================
+ Coverage     78.21%   78.22%   +0.01%     
+ Complexity     2829     2826       -3     
============================================
  Files           465      466       +1     
  Lines          9121     9136      +15     
  Branches       1722     1724       +2     
============================================
+ Hits           7134     7147      +13     
  Misses         1059     1059              
- Partials        928      930       +2     

Impacted Files	Coverage Δ	Complexity Δ
...arturbosch/detekt/formatting/FormattingProvider.kt	100.00% <ø> (ø)	3.00 <0.00> (ø)
...rbosch/detekt/formatting/wrappers/ChainWrapping.kt	100.00% <ø> (ø)	3.00 <0.00> (ø)
...bosch/detekt/formatting/wrappers/CommentSpacing.kt	100.00% <ø> (ø)	3.00 <0.00> (ø)
.../arturbosch/detekt/formatting/wrappers/Filename.kt	100.00% <ø> (ø)	3.00 <0.00> (ø)
...urbosch/detekt/formatting/wrappers/FinalNewline.kt	100.00% <ø> (ø)	4.00 <0.00> (ø)
...ch/detekt/formatting/wrappers/MaximumLineLength.kt	87.50% <ø> (ø)	5.00 <0.00> (ø)
...sch/detekt/formatting/wrappers/ModifierOrdering.kt	100.00% <ø> (ø)	3.00 <0.00> (ø)
...osch/detekt/formatting/wrappers/MultiLineIfElse.kt	100.00% <ø> (ø)	3.00 <0.00> (ø)
...ekt/formatting/wrappers/NoBlankLineBeforeRbrace.kt	100.00% <ø> (ø)	3.00 <0.00> (ø)
...ekt/formatting/wrappers/NoConsecutiveBlankLines.kt	100.00% <ø> (ø)	3.00 <0.00> (ø)
... and 130 more

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 92b6a01...adc79ae. Read the comment docs.

[marschwar]

Just wondering - do we need to still support the kdoc tag as well in order to work for custom rules that have not made the switch?

[cortinico]

Just wondering - do we need to still support the kdoc tag as well in order to work for custom rules that have not made the switch?

To my knowledge, not: the detekt-generator is the only part of the code where we effectively read the KDoc annotations and we don't generated docs for external rules.