🤔 MarcoFalke reviewed a pull request: "ci: Add test coverage job"
(https://github.com/bitcoin/bitcoin/pull/27547#pullrequestreview-1448964603)
I presume there is a setting to disable the comment and just have DrahtBot add a link in the form of `https://app.codecov.io/gh/{owner}/{repo}/pull/{number}` to the summary comment?

💬 MarcoFalke commented on pull request "ci: Add test coverage job":
(https://github.com/bitcoin/bitcoin/pull/27547#discussion_r1208978341)
Also, could move up the task to group it with the other native linux tasks, away from the macos and android task down here?

💬 MarcoFalke commented on pull request "ci: Add test coverage job":
(https://github.com/bitcoin/bitcoin/pull/27547#discussion_r1208977858)
any reason to use lunar, when an Ubuntu LTS or debian bookworm can be used?

💬 MarcoFalke commented on pull request "ci: Add missing set -e to 01_base_install.sh":
(https://github.com/bitcoin/bitcoin/pull/27739#discussion_r1208994069)
I don't think running macOS on Linux is in scope for the CI system.

💬 MarcoFalke commented on pull request "ci: Add missing set -e to 01_base_install.sh":
(https://github.com/bitcoin/bitcoin/pull/27739#discussion_r1208999859)
thx, mentioned in commit description

💬 MarcoFalke commented on pull request "ci: Add missing set -e to 01_base_install.sh":
(https://github.com/bitcoin/bitcoin/pull/27739#discussion_r1209001490)
Also, this was never supported, so seems unrelated to the changes here?

💬 MarcoFalke commented on pull request "refactor: Replace `std::optional<bilingual_str>` with `util::Result`":
(https://github.com/bitcoin/bitcoin/pull/25977#issuecomment-1566720140)
Yeah, that's why I suggested `{std::move()}`, not `std::move()`, see https://github.com/bitcoin/bitcoin/pull/25977#issuecomment-1561270092

💬 kallewoof commented on pull request "test: Throw error when -signetchallenge is non-hex":
(https://github.com/bitcoin/bitcoin/pull/27765#issuecomment-1566738266)
ACK fa6b11a

💬 hebasto commented on pull request "ci: Add missing set -e to 01_base_install.sh":
(https://github.com/bitcoin/bitcoin/pull/27739#discussion_r1209045977)
You are right.

👍 hebasto approved a pull request: "ci: Add missing set -e to 01_base_install.sh"
(https://github.com/bitcoin/bitcoin/pull/27739#pullrequestreview-1449096752)
ACK fa12558d21aa03c22407a1458a0345d8a7ab6a4b

💬 hebasto commented on pull request "ci, iwyu: Update mappings":
(https://github.com/bitcoin/bitcoin/pull/27710#issuecomment-1566747695)
CI failure seems unrelated.

👍 hebasto approved a pull request: "kernel: Remove util/system from kernel library, interface_ui from validation."
(https://github.com/bitcoin/bitcoin/pull/27636#pullrequestreview-1449106502)
re-ACK 7d3b35004b039f2bd606bb46a540de7babdbc41e, only last two commits dropped since my [recent](https://github.com/bitcoin/bitcoin/pull/27636#pullrequestreview-1435394620) review.

💬 TheCharlatan commented on pull request "ci: Add missing set -e to 01_base_install.sh":
(https://github.com/bitcoin/bitcoin/pull/27739#issuecomment-1566766689)
Re https://github.com/bitcoin/bitcoin/pull/27739#issuecomment-1564529747

> Thanks, the last commit should fix your bug.

Thank you for digging into this, seems like everything is working as it should now.
ACK [fa12558](https://github.com/bitcoin/bitcoin/pull/27739/commits/fa12558d21aa03c22407a1458a0345d8a7ab6a4b)

👍 real-or-random approved a pull request: "contrib/init: Better systemd integration"
(https://github.com/bitcoin/bitcoin/pull/25975#pullrequestreview-1449124089)
ACK 689a65d878638ae67b07f111dd77ea3c624e4f58 tested this service file with 25.0

💬 ismaelsadeeq commented on issue "25.0 RC Testing Guide Feedback":
(https://github.com/bitcoin/bitcoin/issues/27736#issuecomment-1566768353)
All comments and suggestions have been addressed, Thank you everyone for the feedback.

💬 real-or-random commented on pull request "Introduce secp256k1 module with field and group classes to test framework":
(https://github.com/bitcoin/bitcoin/pull/26222#discussion_r1209080001)
Perhaps the comment should be explicit about the behavior when `v >= FE.SIZE`

👋 fanquake's pull request is ready for review: "lint: stop ignoring LIEF imports"
(https://github.com/bitcoin/bitcoin/pull/27507)

💬 fanquake commented on pull request "lint: stop ignoring LIEF imports":
(https://github.com/bitcoin/bitcoin/pull/27507#issuecomment-1566839915)
Rebased and updated to use LIEF 0.13.1.

🚀 fanquake merged a pull request: "ci: Add missing set -e to 01_base_install.sh"
(https://github.com/bitcoin/bitcoin/pull/27739)

🚀 fanquake merged a pull request: "test: Throw error when -signetchallenge is non-hex"
(https://github.com/bitcoin/bitcoin/pull/27765)