Comment on DO NOT MERGE

aMockTie@beehaw.org ⁨9⁩ ⁨months⁩ ago

• brisk@aussie.zone ⁨9⁩ ⁨months⁩ ago

  Probably to get some other benefitof the PR system, such as CI tests

  source

  • leds@feddit.dk ⁨9⁩ ⁨months⁩ ago

    Does gerrit have a draft state? In azure devops you can mark PR as draft , won’t trigger any builds but you can still start them manually

    source

    • beeng@discuss.tchncs.de ⁨9⁩ ⁨months⁩ ago

      manually

      This is probably why the dev created a PR, less clicks

      source
    • sf1tzp@programming.dev ⁨9⁩ ⁨months⁩ ago

      I don’t remember if it did when I used it. Our convention was to -2 your own change until it was ready to go 😅

      source
• NotSteve_@lemmy.ca ⁨9⁩ ⁨months⁩ ago

  I do this often. It’s useful if you want to send it to your coworker for some early feedback or as others have said, have the CI run

  source
• DrJenkem@lemmy.blugatch.tube ⁨9⁩ ⁨months⁩ ago

  It’s been a while since I’ve worked with AOSP, but I had always understood it to be some weird shit with Google’s internal processes. The do not merge commits are all over the AOSP, or at least they used to be.

  source