Fix Earthly tasks doing duplicate jobs #829

Author: joepio

Earthfile

@@ -43,16 +43,13 @@ source:
   COPY browser+build/dist /code/browser/data-browser/dist
   DO rust+CARGO --args=fetch
 
-cross-build:
+fmt:
   FROM +source
-  ARG --required TARGET
-  DO rust+SET_CACHE_MOUNTS_ENV
-  WITH DOCKER
-    RUN --mount=$EARTHLY_RUST_CARGO_HOME_CACHE --mount=$EARTHLY_RUST_TARGET_CACHE  cross build --target $TARGET --release
-  END
-  DO rust+COPY_OUTPUT --output=".*" # Copies all files to ./target
-  RUN ./target/$TARGET/release/atomic-server --version
-  SAVE ARTIFACT ./target/$TARGET/release/atomic-server AS LOCAL artifact/bin/atomic-server-$TARGET
+  DO rust+CARGO --args="fmt --check"
+
+lint:
+  FROM +source
+  DO rust+CARGO --args="clippy --no-deps --all-features --all-targets"
 
 build:
   FROM +source
@@ -61,16 +58,19 @@ build:
   SAVE ARTIFACT ./target/release/atomic-server AS LOCAL artifact/bin/atomic-server-x86_64-unknown-linux-gnu
 
 test:
-  FROM +source
+  FROM +build
   DO rust+CARGO --args="test"
 
-fmt:
-  FROM +source
-  DO rust+CARGO --args="fmt --check"
-
-lint:
+cross-build:
   FROM +source
-  DO rust+CARGO --args="clippy --no-deps --all-features --all-targets"
+  ARG --required TARGET
+  DO rust+SET_CACHE_MOUNTS_ENV
+  WITH DOCKER
+    RUN --mount=$EARTHLY_RUST_CARGO_HOME_CACHE --mount=$EARTHLY_RUST_TARGET_CACHE  cross build --target $TARGET --release
+  END
+  DO rust+COPY_OUTPUT --output=".*" # Copies all files to ./target
+  RUN ./target/$TARGET/release/atomic-server --version
+  SAVE ARTIFACT ./target/$TARGET/release/atomic-server AS LOCAL artifact/bin/atomic-server-$TARGET
 
 docker-musl:
   FROM alpine:3.18