The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.

Reviews

See the guideline for information on the review process.

If your review is incorrectly listed, please react with 👎 to this comment and the bot will ignore it on the next update.

NACK. There's no need to install an old version of Python.

updated :

brew install python@3

https://github.com/bitcoin/bitcoin/blob/b2f2951e51b0be370041ec63527bad8fc0dab3ad/doc/build-osx.md#test-suite-dependencies

NACK - python and python@3 are the same thing (both aliases for python 3.11, which is the current latest version). We can leave these docs as-is, and/or worry about changing them when Python 4 is (never) released. Thanks.

To maintain consistency across macOS x86 and Arm64 maybe the minimum python3 version should be [email protected]

ref: #26226

@RandyMcMillan maybe make a separate issue about M1 not supporting Python 3.7 (I don't have one, so can't confirm). It's just one of many operating systems we support, but a lot of devs use it so it seems worth looking into.

Thanks, but there is no need to make an issue. There is nothing here that needs to be solved.