Róbert Papp

This issue was closed in https://github.com/detekt/detekt/pull/5009 released in 1.21.0, and you can see the PR is updating to that version, yet still flagging when I removed the suppression. (screenshot) Since...

Still want to do this, just lower on my priority list. I guess since @BraisGabin is averse to hosting the solution in this repo, we can close this as "won't...

@BraisGabin I would re-open this, there's 3 more rules to write :)

For the record: we've decided to keep the `styles` ruleset for `BracesOn` rules, because their naming already groups them somewhat.

Assuming that you know the base branch of the PR, which is usually the default branch, instead of `fetch-depth: 0`, a more targeted (faster) workaround is: ``` git fetch origin...

@JLLeitschuh how can this be tested with automation? Because contracts should induce Kotlin compilation failures. I guess the positive cases of smart casts as results of contract can be tested:...

I wondered if the that error is suppressed will it emit 1.4 compatible bytecode for the contract? Therefore making it possible to use the same jar binary from both 1.3...

Ran into this again with a Selenium lazy page object: ```kotlin class ScheduleMovieItem( private val root: WebElement ) { val startTime: WebElement get() = root.findElement(By.className("film-start")) val endTime: WebElement get() =...

The problem is `Mockable.addBlockingStub` prepends, why? I guess the answer is to support: ```kotlin every { mockI.f() }.returns(1) // ignored every { mockI.f() }.returns(2) assertEquals(2, mockI.f()) ``` which makes sense,...

Workaround: instead of ``` every { mapper.map(any()) }.returnsMany(obj1, obj2) every { mapper.map(any()) }.throwsMany(ex) ``` use: ``` every { mapper.map(any()) }.invokesMany({ throw ex }, { obj1 }, { obj2 }) ```...