< bitcoin-git> [bitcoin] fanquake opened pull request #21498: refactor: return std::nullopt instead of {} (master...return_std_nullopt) https://github.com/bitcoin/bitcoin/pull/21498
< bitcoin-git> [bitcoin] fanquake pushed 2 commits to master: https://github.com/bitcoin/bitcoin/compare/d2a78ee9288e...80cb51cc6b41
< bitcoin-git> bitcoin/master 7e34448 Jon Atack: test: remove duplicate assertions in util_tests
< bitcoin-git> bitcoin/master 80cb51c fanquake: Merge #21491: test: remove duplicate assertions in util_tests
< bitcoin-git> [bitcoin] fanquake merged pull request #21491: test: remove duplicate assertions in util_tests (master...remove-duplicate-assertions-in-util-tests) https://github.com/bitcoin/bitcoin/pull/21491
< bitcoin-git> [bitcoin] instagibbs closed pull request #16972: Add test for send{toaddress, many} conf_target under subtractfromamount scenario (master...sendtoaddr_conftarget_test) https://github.com/bitcoin/bitcoin/pull/16972
< bitcoin-git> [bitcoin] MarcoFalke pushed 2 commits to master: https://github.com/bitcoin/bitcoin/compare/80cb51cc6b41...786654aa5e0b
< bitcoin-git> bitcoin/master 5294f0d fanquake: refactor: return std::nullopt instead of {}
< bitcoin-git> bitcoin/master 786654a MarcoFalke: Merge #21498: refactor: return std::nullopt instead of {}
< bitcoin-git> [bitcoin] MarcoFalke merged pull request #21498: refactor: return std::nullopt instead of {} (master...return_std_nullopt) https://github.com/bitcoin/bitcoin/pull/21498
< bitcoin-git> [bitcoin] apitko opened pull request #21499: Update bitcoin.conf (master...patch-1) https://github.com/bitcoin/bitcoin/pull/21499
< fanquake> It doesn't matter too much in this case (#21499), but this is kind of annoying: https://0bin.net/paste/PnCrrPU9#hPaZZ9MSIJyHlBFQsxztN+bwGCwx45GkMaJUg5-+Zq0
< gribble> https://github.com/bitcoin/bitcoin/issues/21499 | Update bitcoin.conf by apitko · Pull Request #21499 · bitcoin/bitcoin · GitHub
< fanquake> If a GitHub user with a "too new" account opens a PR, the CI wont run
< fanquake> Nice. I wonder if they'll just start after the 8 hours is up
< bitcoin-git> [bitcoin] MarcoFalke pushed 2 commits to master: https://github.com/bitcoin/bitcoin/compare/786654aa5e0b...1e4a3c057a9b
< bitcoin-git> bitcoin/master fa4ceba MarcoFalke: util: Make Assume() usable as unary expression
< bitcoin-git> bitcoin/master 1e4a3c0 MarcoFalke: Merge #21317: util: Make Assume() usable as unary expression
< bitcoin-git> [bitcoin] MarcoFalke merged pull request #21317: util: Make Assume() usable as unary expression (master...2102-utilAssumeUnary) https://github.com/bitcoin/bitcoin/pull/21317
< wumpus> "If a GitHub user with a "too new" account opens a PR, the CI wont run" interesting, is that an intentional security measure of some kind or a matter of users being synchronized between services only at a delayed pace (i'd guess the latter)
< aj> wumpus: see the link, users were setting up new accounts and using the CI to mine crypto apparently
< wumpus> l-lol
< wumpus> cpu mining was a mistake—
< vasild> MarcoFalke: maybe ConsumeEnumOrRandom() better describes it? wrt https://github.com/bitcoin/bitcoin/pull/21487#issuecomment-803631252
< bitcoin-git> [bitcoin] S3RK opened pull request #21500: wallet, rpc: add an option to list private descriptors (master...listdescriptors_private) https://github.com/bitcoin/bitcoin/pull/21500
< bitcoin-git> [bitcoin] jamesob opened pull request #21502: test: remove unicode from tmp test_runner dir (master...2021-03-testrunner-tmp-no-unicode) https://github.com/bitcoin/bitcoin/pull/21502
< bitcoin-git> [bitcoin] laanwj pushed 2 commits to master: https://github.com/bitcoin/bitcoin/compare/1e4a3c057a9b...c46f1ce75196
< bitcoin-git> bitcoin/master 663f6cd Wladimir J. van der Laan: contrib: Use -daemonwait in systemd init script
< bitcoin-git> bitcoin/master c46f1ce Wladimir J. van der Laan: Merge #21418: contrib: Make systemd invoke dependencies only when ready
< bitcoin-git> [bitcoin] laanwj merged pull request #21418: contrib: Make systemd invoke dependencies only when ready (master...2021-03-systemd-daemonwait) https://github.com/bitcoin/bitcoin/pull/21418
< bitcoin-git> [bitcoin] hebasto opened pull request #21505: build: Remove unused header from the build system (master...210322-miniwget) https://github.com/bitcoin/bitcoin/pull/21505
< hebasto> review begging for one-line bugfix #21344
< gribble> https://github.com/bitcoin/bitcoin/issues/21344 | build: Fix libbitcoinconsensus cross-compiling for Windows with DEBUG=1 by hebasto · Pull Request #21344 · bitcoin/bitcoin · GitHub
< luke-jr> hebasto: see e0077de5de538dd51b5dbd04e05c998d40b20b30
< hebasto> luke-jr: thanks, looking now
< luke-jr> hebasto: I'm not sure what the correct fix is, but I suspect it might be an issue with the pthread pkgconf
< bitcoin-git> [bitcoin] jamesob closed pull request #21502: test: remove unicode from tmp test_runner dir (master...2021-03-testrunner-tmp-no-unicode) https://github.com/bitcoin/bitcoin/pull/21502
< bitcoin-git> [bitcoin] hebasto closed pull request #21344: build: Fix libbitcoinconsensus cross-compiling for Windows with DEBUG=1 (master...210302-debug) https://github.com/bitcoin/bitcoin/pull/21344
< luke-jr> hrm, what to do about C++11-specific bugs in 0.21?
< luke-jr> seems like fixing it would be not worth the effort.
< bitcoin-git> [bitcoin] jonatack opened pull request #21506: p2p, refactor: make NetPermissionFlags an enum class (master...NetPermissionFlags-enum-class) https://github.com/bitcoin/bitcoin/pull/21506
< bitcoin-git> [bitcoin] achow101 opened pull request #21507: Implement BIP8 lockinontimeout (master...bip8) https://github.com/bitcoin/bitcoin/pull/21507
< bitcoin-git> [bitcoin] mzumsande opened pull request #21509: p2p: Don't send FEEFILTER in blocksonly mode. (master...202103_blocksonly_feefilter) https://github.com/bitcoin/bitcoin/pull/21509