Make cargo fmt CI job useful

[dtolnay]

Previously it would only fail if rustfmt crashes on the code in this repo, which is not what I think it was intended to test. Using --check means the job will signal when the code being committed is not correctly formatted.

Separately I removed the git submodule update --init --recursive from the fmt job because it is unnecessary and slow.

[brson]

Thanks!