Everyone disliked that.

  • PorkrollPosadist [he/him, they/them]@hexbear.net
    link
    fedilink
    English
    arrow-up
    8
    ·
    edit-2
    12 days ago

    From my (admittedly, limited) experience, sign-offs are often relatively shallow sanity checks. Nothing about this patch looks egregious? It solves a known problem? It makes it though the CI pipeline? Approved. When dealing with languages like C, where very subtle mistakes can introduce defects and vulnerabilities, I would not trust a LLM to do the brunt of the due diligence which would ordinarily be coming from the contributor (who typically spends a lot more time thinking about the problem than the person signing off on the patch). I’ll admit this isn’t a novel problem, but the amount of scrutiny applied to submissions will definitely need to increase if this becomes a standard process.