| 1 Aug 2025 |
 raitobezarius (DECT: 7248) | In reply to @weethet:catgirl.cloud
Could we just pull the plug and reformat the whole database? Nope | 13:47:20 |
| raitobezarius (DECT: 7248) | In reply to @weethet:catgirl.cloud
Is clang-format not enforced? I have format-on-save on and have run into code changing multiple times by now You should consider using *git-*clang-format | 13:47:40 |
 WeetHet | Wouldn't I end up with the code that doesn't match the code style of the rest of the database? | 13:49:57 |
| WeetHet | Which is even worse | 13:50:07 |
| raitobezarius (DECT: 7248) | No, the goal is to have progressive reformatting of the codebase | 13:50:34 |
| raitobezarius (DECT: 7248) | Once every pieces have been touched, the codebase will be formatted. | 13:50:44 |
 Charles | Fun fact, clang-format formatting is not stable between versions | 13:54:59 |
| 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 |
