06:58:17 Was going through monero github repos and noticed kovri is set as public archive yet kovri-docs and kovri-site aren't 07:01:45 kastelo should be archived too? 07:02:30 @thomasbuilds:matrix.org: Kastelo lol 07:04:34 id argue that kastelo should be migrated to msvb's personal repo and deleted from monero-project 07:24:04 Yeah wherever it's about deleting or archiving it's not up to me. But yeah I'd delete it. 07:25:57 https://mrelay.p2pool.observer/m/matrix.org/MmmNQTxPChqvIMbkNumnURRi.png (Screenshot from 2026-07-22 09-25-30.png) 07:25:58 should also pin gitan sigs repo 07:36:34 .merges 07:36:34 10747 10823 10863 10876 10894 10921 10:22:48 .merge+ 9828 10:22:48 Added 12:04:30 Msg 14:29:33 .merge+ 10896 14:29:33 Added 16:08:07 .merge+ 10358 16:08:07 Added 16:17:00 .merges 16:17:00 10747 10863 10876 10894 10921 19:05:28 .merges 19:05:28 10747 10876 10894 19:13:54 .merge+ 10694 19:13:54 Added 19:47:19 is it preferred to re-indent 100+ lines, or is it okay to add 3 lines without correct indentation for a patch? 19:52:23 The 100 lines are incorrect and the 3 lines match it? 19:52:51 found a different solution anyway 19:54:25 no my question was something else, current code is correct but my patch would add new { } which would change indentation 19:54:37 but i found a solution now without adding { } 19:58:07 Is that code for... goto ? :D 19:59:00 I'm pretty sure I already made at least one patch doing just that (adding extra test and {} but leaving existing code as is without reindent). I typically add extra newlines at the { and } places to make things clear. 100+ lines reindent is way too large and will cause pain and conflicts. I sometimes do the reindent in TF but only because I'm the only one working on it. It's inconsiderate 19:59:06 otherwise. Though I'm sure some will say it's inconsiderate to go the other way. 20:06:59 It's a trade-off for pain now and pain later. No re-indent makes the code much harder to read and maintain. Re-indenting now means the git blame is messed up, and requires a reviewer to make sure that the lines are copy and pasted correctly. 20:08:19 As someone who is updating a decent amount of code, and keeps stumbling upon indentation inconsistencies, I'm really starting to prefer just doing the indentation correct now 20:08:59 no goto, it just has a bit longer lifetime now but should not matter 20:10:20 BTW I'd like to review beg for #9827, a pre-req for carrot_core 20:10:41 I agree with jeffro, also git blame -w exists 20:13:05 And git diff -b