05:15:35 JossieEtheridge opened pull request #10990: Fix long-standing 'Next span is not empty' sync stall (#8194) 05:15:36 > https://github.com/monero-project/monero/pull/10990 05:25:17 omg this PR wtf 05:43:37 I also think they hit that bug because they ser block-sync-size-100 05:43:37 "Measured real, successful 100-block span completions on this node taking 05:43:37 2-26 seconds each. With a 2-second threshold, any peer that goes briefly 05:43:37 quiet — completely normal under real network conditions — gets its[... more lines follow, see https://mrelay.p2pool.observer/e/uKH9hZsLSHVET0Nx ] 05:44:13 block-sync-size=100* 06:30:50 his LLM is so bad he didn't realize he's fixing a closed issue lmao 06:38:51 What I don't understand from github is that it included 20 participants in this PR and there are 0 reviews/comments. 06:38:51 20 ppl received notification for this slop PR rip 06:39:57 @thomasbuilds:matrix.org: Fixong like 5 closed issues from v0.12 06:40:34 aNd fixing a self-induced issue 06:40:50 And pushing release code to master 06:41:11 And not checking if fix is already on master 06:41:17 lol I mean he joined github in April what can you expect 06:42:05 this is his 1st ever contribution 06:43:21 @thomasbuilds:matrix.org: I only see 3 06:43:30 now yes because he forced pushed 06:43:57 @thomasbuilds:matrix.org: Why? He's fixinf a non-issue 06:44:17 he doesn't get it 06:45:58 botched rebase too 06:46:14 Re-added variables that arent used anymore 06:46:27 One of which causes his runawayspans issue 06:47:19 (*thomas licking his lips at dead code) 06:47:27 lmao 06:56:37 May I review beg on https://github.com/monero-project/monero/pull/10934? 07:00:35 thomasbuilds opened pull request #10991: cryptonote_core: remove dead code 07:00:36 > https://github.com/monero-project/monero/pull/10991 07:44:28 10934 is huge, I would review it but I don't have time today 09:30:35 plowsof opened issue #10992: wallet2_api: `doInit()` discards `use_ssl` (onions still attempt TLS h... 09:30:36 > https://github.com/monero-project/monero/issues/10992 11:45:35 thomasbuilds opened pull request #10993: wallet2_api: skip the TLS handshake on Tor/I2P daemons 11:45:36 > https://github.com/monero-project/monero/pull/10993 13:08:09 Looking for US junior developers to join our team! Fully remote position with room for professional growth and solid income potential. Interested? Feel free to shoot me a DM or connect with me on Linkedin: https://www.linkedin.com/in/brando-martinez-b23833b7! 13:39:58 Well, how about solid income instead of merely solid income potential :) 13:41:23 @rbrunner7: That's a potentiality 14:58:45 I got some insane review comments yesterday > <@thomasbuilds:matrix.org> his LLM is so bad he didn't realize he's fixing a closed issue lmao 14:59:18 https://github.com/monero-project/monero/pull/10976#pullrequestreview-4792381477 14:59:18 https://github.com/monero-project/monero/pull/10984#pullrequestreview-4792381594 15:31:28 .merge+ 10951 10952 10950 10949 15:31:28 Added 15:53:55 .merges 15:53:55 10747 10876 10949 10950 19:28:54 .merge+ 10459 19:28:54 Added 19:48:34 typo? 10459 is already merged 19:49:25 I think he meant 10458 19:54:13 .merge+ 10458 19:54:13 Added 20:37:51 ^ Has two approvals now, just FYI 21:15:35 jpk68 opened pull request #10994: chore: consolidate anonymity networks info 21:15:36 > https://github.com/monero-project/monero/pull/10994 22:30:35 jeffro256 opened pull request #10995: wallet: upstream misc stuff before FCMP++ 22:30:36 > https://github.com/monero-project/monero/pull/10995 22:31:17 ^ this PR is a fairly simply review, and gets some churn out of the way before the Carrot/FCMP++ integration reviews 22:31:33 *fairly simple