Thread
#detekt
    a

    Adam

    2 years ago
    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 ")"
    .
    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)
    .
    json(
        """
        {
            "array": [
                ${function(arg1, arg2, arg3)}
            ]
        }
        """.trimIndent()
    )
    Related to this ktlint PR?
    b

    Brais Gabin

    2 years ago
    Yes, it seems like a false positive in ktlint. You can open an issue there with this snippet
    a

    Adam

    2 years ago
    Thanks. I commented on the PR, but if it doesn’t get a response I’ll open an issue.