In commit 9730288a0cd3f33021ef00fb2d95e5216d10ab61 many configs were removed from the CI without explanation.
Fix it by adding them back.
Can be reviewed by looking at:
The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.
For detailed information about the code coverage, see the test coverage report.
See the guideline for information on the review process.
| Type | Reviewers |
|---|---|
| ACK | fanquake |
If your review is incorrectly listed, please react with 👎 to this comment and the bot will ignore it on the next update.
71@@ -72,7 +72,7 @@ jobs:
72 run: |
73 # Run tests on commits after the last merge commit and before the PR head commit
74 # Use clang++, because it is a bit faster and uses less memory than g++
75- git rebase --exec "echo Running test-one-commit on \$( git log -1 ) && CC=clang CXX=clang++ cmake -B build -DBUILD_GUI=ON -DBUILD_BENCH=ON -DBUILD_FUZZ_BINARY=ON -DWITH_BDB=ON -DWITH_NATPMP=ON -DWITH_MINIUPNPC=ON -DWITH_USDT=ON && cmake --build build -j $(nproc) && ctest --test-dir build -j $(nproc) && ./build/test/functional/test_runner.py -j $(( $(nproc) * 2 ))" ${{ env.TEST_BASE }}
76+ git rebase --exec "echo Running test-one-commit on \$( git log -1 ) && CC=clang CXX=clang++ cmake -B build -DWITH_ZMQ=ON -DBUILD_GUI=ON -DBUILD_BENCH=ON -DBUILD_FUZZ_BINARY=ON -DWITH_BDB=ON -DWITH_NATPMP=ON -DWITH_MINIUPNPC=ON -DWITH_USDT=ON && cmake --build build -j $(nproc) && ctest --test-dir build -j $(nproc) && ./build/test/functional/test_runner.py -j $(( $(nproc) * 2 ))" ${{ env.TEST_BASE }}
libzmq3-dev ?
python3-zmq as a proxy for that, which should probably be enough, no? And if libzmq was missing, the CI would be failing, no?
many configs were removed from the CI without explanation.
Nice catch. It’s not clear why a bunch of CI configs would have been removed..
many configs were removed from the CI without explanation.
Nice catch. It’s not clear why a bunch of CI configs would have been removed..
Sorry for overlooking them.
many configs were removed from the CI without explanation.
Nice catch. It’s not clear why a bunch of CI configs would have been removed..
Sorry for overlooking them.
No worries. It is my fault for not reviewing them properly before. Also, it looks like the other reviewers missed them as well, so not your fault again.