Some improvements to the earthly build

Author: Jaykul

.earthlyignore

@@ -1,5 +1,22 @@
+#.git/
+.vscode/
+.github/
+.gitignore
+.editorconfig
 # ignore Modules (because we output there now)
-/Modules/*
-# These aren't part of the module (yet)
-/PotentialContribution/*
-/Content/*
+Modules/
+# For best performance, don't include README etc., to avoid cache-busting
+ReadMe.md
+LICENCE
+# Legacy build files
+build.ps1
+test.ps1
+Install-RequiredModule.ps1
+
+# These aren't part of the actual module
+Templates/
+PotentialContribution/
+Content/
+Output/
+obj/
+bin/

Earthfile

@@ -14,6 +14,19 @@ ARG --global TEMP_ROOT=/temp
 ARG --global MODULE_NAME=ModuleBuilder
 ARG --global CONFIGURATION=Release
 
+# This works on Linux, and speeds things up dramatically because I don't need my .git folder
+# But "LOCALLY" doesn't work on Windows (yet), and that's my main dev environment
+# version:
+#     COPY --if-exists GitVersion.yml .
+#     IF [ -f ./GitVersion.yml ]
+#     ELSE
+#         LOCALLY
+#         COPY tasks+tasks/tasks/GitVersion.yml .
+#     END
+#     LOCALLY
+#     RUN dotnet tool update GitVersion.Tool --version 5.12.0 --global && \
+#         dotnet gitversion > version.json
+#     SAVE ARTIFACT ./version.json
 
 worker:
     # Dotnet tools and scripts installed by PSGet
@@ -26,24 +39,27 @@ worker:
     # Dealing with dependencies first allows docker to cache packages for us
     # So the dependency cach only re-builds when you add a new dependency
     COPY RequiredModules.psd1 .
-    # COPY *.csproj .
+    # COPY --if-exists *.csproj .
     RUN ["pwsh", "-File", "/Tasks/_Bootstrap.ps1", "-RequiredModulesPath", "RequiredModules.psd1"]
 
 build:
     FROM +worker
     RUN mkdir $OUTPUT_ROOT $TEST_ROOT $TEMP_ROOT
-    COPY . .
-    # make sure you have bin and obj in .earthlyignore, as their content from context might cause problems
+    # On Linux we could use the version: task, we wouldn't need .git/ here and
+    # we could avoid re-running this every time there was a commit
+    # we could copying the Source and Invoke-Build script to improve caching
+    COPY --dir .git/ Source/ build.psd1 Build.build.ps1 GitVersion.yml nuget.config /work
     RUN ["pwsh", "-Command", "Invoke-Build", "-Task", "Build", "-File", "Build.build.ps1"]
 
     # SAVE ARTIFACT [--keep-ts] [--keep-own] [--if-exists] [--force] <src> [<artifact-dest-path>] [AS LOCAL <local-path>]
     SAVE ARTIFACT $OUTPUT_ROOT/$MODULE_NAME AS LOCAL ./Modules/$MODULE_NAME
 
 test:
     # If we run a target as a reference in FROM or COPY, it's outputs will not be produced
-    BUILD +build
+    # BUILD +build
     FROM +build
-    # make sure you have bin and obj in .earthlyignore, as their content from context might cause problems
+    # Copy the test files here, so we can avoid rebuilding when iterating on tests
+    COPY --dir Tests/ ScriptAnalyzerSettings.psd1 /work
     RUN ["pwsh", "-Command", "Invoke-Build", "-Task", "Test", "-File", "Build.build.ps1"]
 
     # SAVE ARTIFACT [--keep-ts] [--keep-own] [--if-exists] [--force] <src> [<artifact-dest-path>] [AS LOCAL <local-path>]