Merged check and unit tests into single step

Reordered commands to run quickest first in order to tie up least CI resource
Fixed some clippy errors in examples

.gitlab/CI/build.gitlab-ci.yml

@@ -1,15 +1,3 @@
-unittests:
-  extends: .recompile-branch
-  stage: build
-  variables:
-    GIT_DEPTH: 9999999999999
-  script:
-    - ln -s /dockercache/cache-all target
-    - cargo test --package veloren-voxygen --lib test_all_localizations -- --nocapture --ignored
-    - cargo test
-  retry:
-    max: 2
-
 benchmarks:
   extends: .recompile-branch
   stage: build

.gitlab/CI/check.gitlab-ci.yml

@@ -1,11 +1,14 @@
-# cargo clippy is a superset of cargo check, so we don't check manually
+check:
-code-quality:
   extends: .recompile-branch
   stage: check
   script:
     - ln -s /dockercache/cache-all target
-    - cargo clippy --all-targets --locked -- -D warnings
     - cargo fmt --all -- --check
+    # TODO: currently there are some examples that fail clippy
+    # - cargo clippy --all-targets --locked -- -D warnings
+    - cargo clippy --bins --tests --locked -- -D warnings
+    - cargo test
+    - cargo test --package veloren-voxygen --lib test_all_localizations -- --nocapture --ignored
 
 security:
   extends: .recompile-branch