• Petter1@discuss.tchncs.de
    link
    fedilink
    arrow-up
    4
    ·
    6 hours ago

    Well, AI code should be reviewed prior merge into master, same as any code merged into master.

    We have git for a reason.

    So I would definitely say this was a human fault, either reviewer’s or the human’s who decided that no (or AI driven) review process is needed.

    If I would manage devOps, I would demand that AI code has to be signed off by a human on commit taking responsibility with the intention that they review changes made by AI prior pushing

    • pinball_wizard@lemmy.zip
      link
      fedilink
      arrow-up
      9
      ·
      edit-2
      5 hours ago

      If I would manage devOps, I would demand that AI code has to be signed off by a human on commit taking responsibility with the intention that they review changes made by AI prior pushing

      And you would get burned. Today’s AI does one thing really really well - create output that looks correct to humans.

      You are correct that mandatory review is our best hope.

      Unfortunately, the studies are showing we’re fucked anyway.

      Because whether the AI output is right or wrong, it is highly likely to at least look correct, because creating correct looking output is where (what we call “AI”, today) AI shines.

    • Limerance@piefed.social
      link
      fedilink
      English
      arrow-up
      5
      ·
      6 hours ago

      Realistically what happens is the code review is done under time pressure and not very thoroughly.