Replies: 1 comment 1 reply
-
|
That's similar to what Toshiaki did here: https://github.com/detekt/detekt/pull/8436/files#diff-2fefb6430b5a3c1a148a97da349e0481f60d3ff6c72535f44fd9ca360b3a8830 If you need that you can take his diff. That PR is a bit stuck. |
Beta Was this translation helpful? Give feedback.
1 reply
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
-
For this I tried to write TC like below
here
toLowerCase()is annotated with@DeprecatedSinceKotlin(warningSince = "1.5", errorSince = "2.1")so it should not give any compilation error but when I run the TC I get the errore: file:///Users/username/detekt/detekt-rules-errorprone/src/test/kotlin/dev/detekt/rules/bugs/DeprecationSpec.kt:193:8 'fun String.toLowerCase(): String' is deprecated. Use lowercase() instead.I have updated the
lintWithContextlike below to pass thelanguageVersionSettingsto complier configuration but it still gives the above errorI am not if there is some other way to achieve this
context: It used to pass earlier with the code mentioned in the #4084 (comment) but now it doesn't. Not sure is there is some other way(or already a tracking issue) if not then I can create an issue
Beta Was this translation helpful? Give feedback.
All reactions