| 1 Aug 2025 |
Charles | So if you only do it piecemeal then your formatting will probably never actually converge | 13:55:41 |
piegames | Something something eventually consistent | 13:59:23 |
helle (just a stray cat girl) | I love mdbook's look and result, I hate how it gets there..... sigh I may be looking at ways of long term fixing this, so the patchwork of filters that work on raw markdown strings with inconsistencies can go away and I will need to remember to file a bug or two about this | 14:11:19 |
K900 | How the fuck am I code owner | 14:17:49 |
K900 | lol | 14:17:51 |
raitobezarius (DECT: 7248) | In reply to @charles:computer.surgery So if you only do it piecemeal then your formatting will probably never actually converge even a better reason for not doing in one shot then | 14:18:02 |
raitobezarius (DECT: 7248) | because if CI enforces absolute validity, then we will have to reformat on each bumps | 14:18:11 |
raitobezarius (DECT: 7248) | which is a bit ridiculous | 14:18:14 |
Charles | huh? that means you have to do it in one shot, and then reformat on bumps | 14:18:21 |
raitobezarius (DECT: 7248) | In reply to @k900:0upti.me How the fuck am I code owner well you received an email about this cc jade_ :p | 14:18:25 |
Charles | yeah that's not a huge deal i don't think | 14:18:26 |
raitobezarius (DECT: 7248) | i am a big hater of "format commits" | 14:18:47 |
raitobezarius (DECT: 7248) | no useful information are contained in there | 14:18:55 |
Charles | i think i would hate inconsistent formatting more | 14:19:15 |
raitobezarius (DECT: 7248) | i personally don't care | 14:19:26 |
Charles | see also .git-blame-ignore-revs | 14:19:26 |
raitobezarius (DECT: 7248) | yes but it only ignore them on git blame | 14:19:37 |
raitobezarius (DECT: 7248) | not on git rebase, not on git bisect, not on […] | 14:19:46 |
emily | fwiw I would be so much happier if it was just done outright | 16:03:58 |
emily | and the conflicts dealt with ~once | 16:04:04 |
emily | because the partial formatting is inconvenient for My Workflow™ | 16:04:14 |
emily | (but I suspect others' workflow too, in general partial file formatting is not that great in terms of LSP support etc.) | 16:04:29 |
raitobezarius (DECT: 7248) | but aren't editors these days able to have project-local configuration? | 16:04:57 |
emily | Nix is doing (already did?) a treewide format | 16:04:58 |
emily | or do you mean across stable versions? | 16:05:05 |
raitobezarius (DECT: 7248) | In reply to @emilazy:matrix.org or do you mean across stable versions? Yes | 16:05:29 |
helle (just a stray cat girl) | (cross porting patches can be hilarious at times anyway, but yeah, it would make it more impossible) | 16:05:54 |
emily | can just format on all branches | 16:05:45 |
emily | it's not about per-project config, it's about having to do region-based formats rather than whole-file ones etc. | 16:06:11 |
raitobezarius (DECT: 7248) | In reply to @emilazy:matrix.org it's not about per-project config, it's about having to do region-based formats rather than whole-file ones etc. That's the job of the pre-commit hook though, no? | 16:06:26 |