Skip to content

Commit 3ec073d

Browse files
committed
cleanup
1 parent 198c91c commit 3ec073d

13 files changed

+19
-13
lines changed

build/artifacts/BuildContext.cs

+1-1
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@ namespace Artifacts;
44

55
public class BuildContext : BuildContextBase
66
{
7-
public string MsBuildConfiguration { get; set; } = "Release";
7+
public string MsBuildConfiguration { get; set; } = Constants.DefaultConfiguration;
88

99
public bool IsDockerOnLinux { get; set; }
1010

build/artifacts/BuildLifetime.cs

+1
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,4 @@
1+
using Common.Lifetime;
12
using Common.Utilities;
23

34
namespace Artifacts;

build/build/BuildContext.cs

+1-1
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@ namespace Build;
55

66
public class BuildContext : BuildContextBase
77
{
8-
public string MsBuildConfiguration { get; set; } = "Release";
8+
public string MsBuildConfiguration { get; set; } = Constants.DefaultConfiguration;
99

1010
public readonly Dictionary<PlatformFamily, string[]> NativeRuntimes = new()
1111
{

build/build/BuildLifetime.cs

+2-1
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,5 @@
11
using Build.Utilities;
2+
using Common.Lifetime;
23
using Common.Utilities;
34

45
namespace Build;
@@ -9,7 +10,7 @@ public override void Setup(BuildContext context, ISetupContext info)
910
{
1011
base.Setup(context, info);
1112

12-
context.MsBuildConfiguration = context.Argument(Arguments.Configuration, "Release");
13+
context.MsBuildConfiguration = context.Argument(Arguments.Configuration, Constants.DefaultConfiguration);
1314
context.EnabledUnitTests = context.IsEnabled(EnvVars.EnabledUnitTests);
1415

1516
context.Credentials = Credentials.GetCredentials(context);

build/build/Utilities/ContextExtensions.cs

-7
This file was deleted.

build/common/Utilities/BuildLifetimeBase.cs renamed to build/common/Lifetime/BuildLifetimeBase.cs

+2-1
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,8 @@
11
using Cake.Incubator.LoggingExtensions;
22
using Common.Addins.GitVersion;
3+
using Common.Utilities;
34

4-
namespace Common.Utilities;
5+
namespace Common.Lifetime;
56

67
public class BuildLifetimeBase<T> : FrostingLifetime<T> where T : BuildContextBase
78
{

build/common/Utilities/Constants.cs

+3
Original file line numberDiff line numberDiff line change
@@ -11,6 +11,9 @@ public class Constants
1111
public const string NetVersion60 = $"net{Version60}";
1212
public const string NetVersion70 = $"net{Version70}";
1313

14+
public const string DefaultBranch = "main";
15+
public const string DefaultConfiguration = "Release";
16+
1417
public static readonly string[] VersionsToBuild = { Version60, Version70 };
1518
public static readonly Architecture[] ArchToBuild = { Architecture.Amd64, Architecture.Arm64 };
1619
public static readonly string[] DistrosToSkip = { Alpine313, Alpine314, Centos7, Ubuntu2204 };

build/common/Utilities/ContextExtensions.cs

+4-1
Original file line numberDiff line numberDiff line change
@@ -52,7 +52,7 @@ public static bool IsOriginalRepo(this ICakeContext context)
5252
public static bool IsMainBranch(this ICakeContext context)
5353
{
5454
var repositoryBranch = GetBranchName(context);
55-
return !string.IsNullOrWhiteSpace(repositoryBranch) && StringComparer.OrdinalIgnoreCase.Equals("main", repositoryBranch);
55+
return !string.IsNullOrWhiteSpace(repositoryBranch) && StringComparer.OrdinalIgnoreCase.Equals(Constants.DefaultBranch, repositoryBranch);
5656
}
5757

5858
public static bool IsSupportBranch(this ICakeContext context)
@@ -163,4 +163,7 @@ public static string GetBranchName(this ICakeContext context)
163163
private static void StartGroup(this IAzurePipelinesCommands _, ICakeContext context, string title) => context.Information("##[group]{0}", title);
164164

165165
private static void EndGroup(this IAzurePipelinesCommands _, ICakeContext context) => context.Information("##[endgroup]");
166+
167+
public static FilePath? GetGitVersionToolLocation(this ICakeContext context) =>
168+
context.GetFiles($"src/GitVersion.App/bin/{Constants.DefaultConfiguration}/{Constants.NetVersion60}/gitversion.dll").SingleOrDefault();
166169
}

build/docker/BuildLifetime.cs

+1
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,4 @@
1+
using Common.Lifetime;
12
using Common.Utilities;
23

34
namespace Docker;

build/docs/BuildLifetime.cs

+1
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,5 @@
11
using Cake.Wyam;
2+
using Common.Lifetime;
23
using Common.Utilities;
34
using Docs.Utilities;
45

build/publish/BuildLifetime.cs

+1
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,4 @@
1+
using Common.Lifetime;
12
using Common.Utilities;
23
using Publish.Utilities;
34

build/release/BuildLifetime.cs

+1
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,4 @@
1+
using Common.Lifetime;
12
using Common.Utilities;
23
using Release.Utilities;
34

build/release/Tasks/PublishRelease.cs

+1-1
Original file line numberDiff line numberDiff line change
@@ -47,7 +47,7 @@ public override void Run(BuildContext context)
4747
Milestone = milestone,
4848
Name = milestone,
4949
Prerelease = false,
50-
TargetCommitish = "main"
50+
TargetCommitish = Constants.DefaultBranch
5151
});
5252

5353
context.GitReleaseManagerAddAssets(token, Constants.RepoOwner, Constants.Repository, milestone, assets);

0 commit comments

Comments
 (0)