I’m seeing lots of results for `ParameterListWrapp...
# detekt
a
I’m seeing lots of results for
ParameterListWrapping
in 1.12.0 that look either questionable or wrong. Avoiding upgrading for now, rather than disabling the rule. A couple of examples: Questionable. Fails with
Argument should be on a separate line (unless all arguments can fit a single line)
and
Missing newline before ")"
.
Copy code
foo(bar.apply {
    // stuff
})
Wrong? Fails
function(arg1, arg2, arg3)
with
Argument should be on a separate line (unless all arguments can fit a single line)
.
Copy code
json(
    """
    {
        "array": [
            ${function(arg1, arg2, arg3)}
        ]
    }
    """.trimIndent()
)
Related to this ktlint PR?
b
Yes, it seems like a false positive in ktlint. You can open an issue there with this snippet
a
Thanks. I commented on the PR, but if it doesn’t get a response I’ll open an issue.
👍 1
131 Views