| 31 Jul 2025 |
emily | well there are test failures on some of the commits but the __structuredAttrs bug prevents the CI log from being useful at all | 22:04:46 |
emily | oh I also didn't request reviewers for the https://gerrit.lix.systems/c/lix/+/3834/7 stack | 22:05:27 |
emily | which would fix that | 22:05:32 |
emily | I'll do it later | 22:05:39 |
emily | (oh, well actually that part got merged… so I have no idea what is going on and will have to look into it later) | 22:06:33 |
raitobezarius | packaging wise | 22:34:48 |
raitobezarius | just put qyriad | 22:34:50 |
raitobezarius | (Qyriad don't hate me :D) | 22:34:54 |
raitobezarius | this can be expedited | 22:34:59 |
emily | unfortunately the error logs should already get fixed so I'm confused :) I'll take a look in a bit though | 22:37:35 |
emily | is there a reason to have codeowner request as an explicit step rather than automatic? | 22:37:51 |
emily | aside from whether the step is CL or stack at once | 22:38:00 |
raitobezarius | i am confused because I'm almost certain there must be an automatic thing | 22:38:32 |
raitobezarius | but maybe Google has a secret plugin | 22:38:41 |
| 1 Aug 2025 |
K900 | @raitobezarius any reason I'm r?'d on those two changes | 04:53:32 |
raitobezarius | In reply to @k900:0upti.me @raitobezarius any reason I'm r?'d on those two changes because you are code owner and the plugin distribute load :P | 13:08:43 |
WeetHet | Is clang-format not enforced? I have format-on-save on and have run into code changing multiple times by now | 13:18:50 |
WeetHet | Could we just pull the plug and reformat the whole database? | 13:19:05 |
piegames | That would make cross-porting patches impossible, unfortunately | 13:33:54 |
WeetHet | In reply to @piegames:flausch.social That would make cross-porting patches impossible, unfortunately Aren't there tools for ast-based merging? | 13:34:47 |
WeetHet | https://mergiraf.org/ | 13:35:02 |
piegames | I've tried out Mergiraf when it was fresh, and I'd say that it's not ready yet | 13:35:33 |
WeetHet | I have it set as my merge tool, it seems okay | 13:36:09 |
raitobezarius | In reply to @weethet:catgirl.cloud Could we just pull the plug and reformat the whole database? Nope | 13:47:20 |
raitobezarius | 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 | No, the goal is to have progressive reformatting of the codebase | 13:50:34 |
raitobezarius | 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 |