Add comments for future improvements

Signed-off-by: lemccomb <[email protected]>

.github/workflows/dotnet.yml

@@ -278,6 +278,9 @@ jobs:
         dotnet publish --configuration Debug --self-contained true --output published/debug --property:FileVersion=$VERSION CoseSignTool/CoseSignTool.csproj
         dotnet publish --configuration Release --self-contained true --output published/release --property:FileVersion=$VERSION CoseSignTool/CoseSignTool.csproj
         # Self-contained is needed. Must use .csproj instead of .sln.
+        # Ideally we should also verify in the Build and Test job, but that will require pre-caulculating the version number and either
+        #   Running build and test separately because we can't pass the version number to dotnet test, or
+        #   Setting the version number dynamically in the csproj files, using <FileVersion>$(VersionBin)</FileVersion>  
       shell: bash
 
     # List the contents of the published directory to make sure all the artifacts are there.