Comments (4)
schalkms
commented on May 25, 2024
1
Thank you for bringing this topic to our attention.
I added a clarifying comment to the detekt rule doc, which prevents further issues like this.
from detekt.
schalkms
commented on May 25, 2024
One rule originates from ktlint wrapped in the formatting ruleset, one from the native style ruleset.
I don't know what detekt should handle here. What do you suggest?
Detekt includes multiple conflicting rules, where the user decides for one or the other rule to be enabled.
from detekt.
mrclrchtr
commented on May 25, 2024
Thank you very much for the quick reply!
Ah ok, I see. I thought that because both are indentSize: 2 that it should work with each other. I don't have a suggested solution.
But maybe it would be useful to document such incompatibilities. I have spent some time looking for references.
The ticket can be closed if no TODO can be derived from your side.
from detekt.
mrclrchtr
commented on May 25, 2024
Thank you very much ๐
from detekt.
Related Issues (20)
- CoroutineLaunchedInTestWithoutRunTest | Fail to detect Flow based Coroutine Launches HOT 1
- Question for new rule to check if test method is missing `@Test` annotation HOT 2
- False negative for companion object in UndocumentedPublicClass
- False negative for enum constants in UndocumentedPublic* rule group HOT 2
- UnusedPrivateProperty does not detect unused property named "logger" HOT 2
- `RedundantSuspendModifier` false-positives with extension functions HOT 3
- False-positve `UnusedImports` when there are mutiple functions of the same name
- Create a property to store the `basePath`
- Alias ObjectPropertyNaming as ObjectPropertyName
- StringLiteralDuplication potential false positive with interpolated strings HOT 3
- NamedArguments: False positive on spread varargs
- Ignore / Suppress specific magic number warning in detekt baseline.xml for a Compose function HOT 1
- detekt-core shouldn't depend on detekt-psi-utils HOT 2
- ignoreFailures doesn't work when using gradle workers
- False positive `SwallowedException` when exception is used as a receiver
- Multiline strings๏ปฟ may occur parse error HOT 3
- "Missing new line after {" is reported incorrectly in nested lambda
- Incorrect report of error - UnsafeOptInUsageError HOT 2
- MatchingDeclarationName - Not detecting method names - Jetpack Compose HOT 10
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
๐ Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. ๐๐๐
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google โค๏ธ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from detekt.