Merge pull request #322 from WildGums/GitHubSync/20210210-095751

GitHubSync update
WildGums · Feb 10, 2021 · d2c69b6 · d2c69b6
2 parents 04eaa12 + 80f04e9
commit d2c69b6
Show file tree

Hide file tree

Showing 7 changed files with 12 additions and 34 deletions.
diff --git a/.github/workflows/cake-buildscripts-dependabot.yml b/.github/workflows/cake-buildscripts-dependabot.yml
diff --git a/deployment/cake/dependencies-tasks.cake b/deployment/cake/dependencies-tasks.cake
@@ -28,10 +28,11 @@ public class DependenciesProcessor : ProcessorBase
         // is required to prevent issues with foreach
         foreach (var dependency in BuildContext.Dependencies.Items.ToList())
         {
-            if (!ShouldProcessProject(BuildContext, dependency))
-            {
-                BuildContext.Dependencies.Items.Remove(dependency);
-            }
+            // Note: dependencies should always be built
+            // if (!ShouldProcessProject(BuildContext, dependency))
+            // {
+            //     BuildContext.Dependencies.Items.Remove(dependency);
+            // }
         }
     }
 

diff --git a/deployment/cake/generic-variables.cake b/deployment/cake/generic-variables.cake
@@ -1,6 +1,6 @@
 #l "buildserver.cake"
 
-#tool "nuget:?package=GitVersion.CommandLine&version=5.6.4"
+#tool "nuget:?package=GitVersion.CommandLine&version=5.6.6"
 
 //-------------------------------------------------------------
 

diff --git a/deployment/cake/github-pages-tasks.cake b/deployment/cake/github-pages-tasks.cake
@@ -1,6 +1,6 @@
 #l "github-pages-variables.cake"
 
-#addin "nuget:?package=Cake.Git&version=0.22.0"
+#addin "nuget:?package=Cake.Git&version=1.0.0"
 
 //-------------------------------------------------------------
 

diff --git a/deployment/cake/lib-generic.cake b/deployment/cake/lib-generic.cake
@@ -518,14 +518,14 @@ private static bool ShouldProcessProject(BuildContext buildContext, string proje
 
     // Experimental mode where we ignore projects that are not on the deploy list when not in CI mode, but
     // it can only work if they are not part of unit tests (but that should never happen)
-    if (buildContext.Tests.Items.Count == 0)
-    {
+    // if (buildContext.Tests.Items.Count == 0)
+    // {
         if (checkDeployment && !ShouldDeployProject(buildContext, projectName))
         {
             buildContext.CakeContext.Warning("Project '{0}' should not be processed because this is not a CI build, does not contain tests and the project should not be deployed, removing from projects to process", projectName);
             return false;
         }
-    }
+    //}
 
     return true;
 }

diff --git a/deployment/cake/lib-octopusdeploy.cake b/deployment/cake/lib-octopusdeploy.cake
@@ -1,4 +1,4 @@
-#tool "nuget:?package=OctopusTools&version=7.4.4"
+#tool "nuget:?package=OctopusTools&version=7.4.6"
 
 public class OctopusDeployIntegration : IntegrationBase
 {

diff --git a/deployment/cake/notifications-msteams.cake b/deployment/cake/notifications-msteams.cake
@@ -1,4 +1,4 @@
-#addin "nuget:?package=Cake.MicrosoftTeams&version=0.9.0"
+#addin "nuget:?package=Cake.MicrosoftTeams&version=1.0.0"
 
 //-------------------------------------------------------------