Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

GitHubSync update #322

Merged
merged 1 commit into from
Feb 10, 2021
Merged

GitHubSync update #322

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
23 changes: 0 additions & 23 deletions .github/workflows/cake-buildscripts-dependabot.yml
View file
Open in desktop

This file was deleted.

9 changes: 5 additions & 4 deletions deployment/cake/dependencies-tasks.cake
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -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);
// }
}
}

Expand Down
2 changes: 1 addition & 1 deletion deployment/cake/generic-variables.cake
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -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"

//-------------------------------------------------------------

Expand Down
2 changes: 1 addition & 1 deletion deployment/cake/github-pages-tasks.cake
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -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"

//-------------------------------------------------------------

Expand Down
6 changes: 3 additions & 3 deletions deployment/cake/lib-generic.cake
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -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;
}
Expand Down
2 changes: 1 addition & 1 deletion deployment/cake/lib-octopusdeploy.cake
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
#tool "nuget:?package=OctopusTools&version=7.4.4"
#tool "nuget:?package=OctopusTools&version=7.4.6"

public class OctopusDeployIntegration : IntegrationBase
{
Expand Down
2 changes: 1 addition & 1 deletion deployment/cake/notifications-msteams.cake
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
#addin "nuget:?package=Cake.MicrosoftTeams&version=0.9.0"
#addin "nuget:?package=Cake.MicrosoftTeams&version=1.0.0"

//-------------------------------------------------------------

Expand Down