All other check-all dependencies are more or less directly related to node development. Including check-electrum adds a bunch of unrelated requirements for python dependencies for node developers. This is not clearly explained in the CONTRIBUTING and build documentation, which implies that the electrum dependencies are only needed for electrum devs.

Let's remove that dependency, and run the check-electrum target only on diffs that touch the electrum subtree and on master commits. There is currently no way a commit that does not touch the electrum subtree can break the electrum unit tests, they are only related to the electrum codebase.

Check that electrum unit tests are not run on check-all.

mkdir build
cd build
cmake .. -GNinja
ninja all check-all
ninja check-electrum