Fix caching of NuGet packages caching (use explicit path, add restore keys and remove restore condition)

Author: ronaldbarendse

azure-pipelines.yml

@@ -1,7 +1,7 @@
 variables:
   solution: Umbraco.StorageProviders.sln
   buildConfiguration: Release
-  NUGET_PACKAGES: ''
+  NUGET_PACKAGES: $(Pipeline.Workspace)/.nuget/packages
   DOTNET_NOLOGO: true
   DOTNET_GENERATE_ASPNET_CERTIFICATE: false
   DOTNET_SKIP_FIRST_TIME_EXPERIENCE: true
@@ -14,15 +14,16 @@ stages:
           vmImage: windows-latest
         steps:
           - task: Cache@2
-            displayName: Cache NuGet packages
             inputs:
               key: 'nuget | "$(Agent.OS)" | **/packages.lock.json'
-              path: '$(NUGET_PACKAGES)'
-              cacheHitVar: 'CACHE_RESTORED'
+              restoreKeys: |
+                nuget | "$(Agent.OS)"
+                nuget
+              path: $(NUGET_PACKAGES)
+            displayName: Cache NuGet packages
 
           - script: dotnet restore $(solution) --locked-mode
             displayName: Restore NuGet packages
-            condition: ne(variables.CACHE_RESTORED, true)
 
           - script: dotnet build $(solution) -c $(buildConfiguration) -p:ContinuousIntegrationBuild=true --no-restore
             displayName: Build
@@ -31,6 +32,6 @@ stages:
             displayName: Pack
 
           - task: PublishBuildArtifacts@1
-            displayName: Publish NuGet packages
             inputs:
                 ArtifactName: nupkg
+            displayName: Publish NuGet packages