| 17 May 2024 |
 Qyriad | (which btw is also in refs/notes/, just not on the commit itself) | 00:36:41 |
 jade_ | i am confused why you can't have that behaviour on merge always | 00:37:02 |
| Qyriad | I guess just since it would then have put to that message in the merge commit and duplicate it for every commit in the merge? idk | 00:37:40 |
 puck | In reply to @jade_:matrix.org
i am confused why you can't have that behaviour on merge always mmm, we can check the source tomorrow | 00:38:05 |
 delroth | fwiw it doesn't add a merge commit if it doesn't need to, see e.g. my 2 commits on main: https://git.lix.systems/lix-project/lix/commits/branch/main | 00:39:32 |
| delroth | * fwiw it doesn't add a merge commit if it doesn't need to, see e.g. my 3 commits on main: https://git.lix.systems/lix-project/lix/commits/branch/main | 00:39:40 |
| Qyriad | okay yeah that's what we thought | 00:40:33 |
| jade_ | In reply to @qyriad:katesiria.org
I guess just since it would then have put to that message in the merge commit and duplicate it for every commit in the merge? idk mm right yeah ok that makes sense | 00:40:54 |
| Qyriad | we could just get in the habit of hitting the rebase button in gerrit more often. most of the time that even keeps reviews | 00:40:55 |
| puck | it probably merges if you have >1 commit | 00:42:24 |
| Qyriad | I think it is probably >1 commit submitted at the same time || commit's base is not HEAD | 00:42:52 |
 julia | you can have more than 1 commit with Gerrit? do I misunderstand how it works | 00:42:55 |
| Qyriad | you can have a relation chain! | 00:43:26 |
| julia | oh those can be merged all at once? | 00:43:44 |
| Qyriad | you can push a commit and that gets a change ID, and then push another commit on top which still gets reviewed separately, but can all be merged at once yep! | 00:44:00 |
| Qyriad | this is one of the best things about gerrit. you can be working on these huuuuuge changes that are still reviewable because they're split up logically and you can review each commit as it comes, but then still keep working on the later parts without blocking the whole thing | 00:45:35 |
| Qyriad | and then if it turns out that the later commits have some problems that will take more time, you can just merge the entire chain below and still have merged significant progress into the codebase | 00:46:19 |
| julia | :) | 00:46:33 |
| jade_ | it also /forces/ good commit hygiene | 00:51:27 |
| jade_ | idk if you have noticed the commit history of lix but it is probably one of the cleanest and clearest commit histories i have ever worked with in any project. | 00:51:48 |
 strawberry🍓 (it/pup/she/they) 🏳️⚧️ 💜 🦴 (puppygock.gay) | In reply to @jade_:matrix.org
it also /forces/ good commit hygiene most underrated thing about projects ever | 00:52:05 |
| strawberry🍓 (it/pup/she/they) 🏳️⚧️ 💜 🦴 (puppygock.gay) | idk if it's just the places i hang out in but apparently it's such an unpopular opinion to want clean commit history | 00:52:30 |
| Qyriad | see, the thing about clean commit history | 00:53:13 |
| jade_ | the thing about gerrit is that it /forces/ it because your commit message is your change description. you have to write something in there for someone to be able to review it and understand where you're coming from | 00:53:35 |
| Qyriad | is it sounds like potential dogma right? it sounds like "yeah this is clearly best practices but shrugs" for a lot of people. but clean commit history doesn't just look nice, and it's not even just nice for rebases, it is necessary for code review | 00:54:46 |
| jade_ | yeah. github having the change description in two places is wild though | 00:55:23 |
| Qyriad | clean commit history is nice for looking back on. but it is necessary to be able to review changes | 00:55:25 |
| jade_ | oh right i think i remember that the forgejo AGit flow doesn't update PR descriptions into line with commit messages | 00:56:00 |
| strawberry🍓 (it/pup/she/they) 🏳️⚧️ 💜 🦴 (puppygock.gay) | i dont even have high standards, i just dont want PRs to be full of "Update <filename>", "address reviews", "Update <filename>, "address more reviews", etc | 00:56:07 |
| strawberry🍓 (it/pup/she/they) 🏳️⚧️ 💜 🦴 (puppygock.gay) | and merged like that | 00:56:16 |
