243
DO NOT MERGE (feddit.dk)

Merged

you are viewing a single comment's thread
view the rest of the comments
[-] aMockTie@beehaw.org 21 points 4 months ago

Why did they submit this as a pull request in the first place? Just commit it to a WIP branch until it’s ready to merge. Am I missing something?

[-] brisk@aussie.zone 30 points 4 months ago

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

[-] leds@feddit.dk 10 points 4 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

[-] beeng@discuss.tchncs.de 4 points 4 months ago

manually

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

[-] sf1tzp@programming.dev 2 points 4 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 😅

load more comments (4 replies)
this post was submitted on 18 Jan 2024
243 points (96.6% liked)

Programmer Humor

17608 readers
491 users here now

Welcome to Programmer Humor!

This is a place where you can post jokes, memes, humor, etc. related to programming!

For sharing awful code theres also Programming Horror.

Rules

founded 11 months ago
MODERATORS