From 550f8829773f18407acfd47a79d725147c4da0c1 Mon Sep 17 00:00:00 2001 From: Tom Walker Date: Tue, 22 Sep 2026 05:30:53 -0400 Subject: [PATCH 1/2] #31 Update to Cake 6.3 --- .config/dotnet-tools.json | 2 +- build.cake | 19 +++++++++---- build/BuildConfig.cake | 53 +----------------------------------- build/PublishOperations.cake | 2 +- 4 files changed, 17 insertions(+), 59 deletions(-) diff --git a/.config/dotnet-tools.json b/.config/dotnet-tools.json index 831d039..022e28e 100644 --- a/.config/dotnet-tools.json +++ b/.config/dotnet-tools.json @@ -3,7 +3,7 @@ "isRoot": true, "tools": { "cake.tool": { - "version": "3.0.0", + "version": "6.3.0", "commands": [ "dotnet-cake" ] diff --git a/build.cake b/build.cake index 0e07777..ead6d6c 100644 --- a/build.cake +++ b/build.cake @@ -1,6 +1,3 @@ -// Addins -#addin nuget:?package=Cake.ArgumentBinder&version=3.0.0 - // Usings using System.IO; using System.Threading.Tasks; @@ -14,7 +11,19 @@ using System.Threading.Tasks; #load "./build/PublishOperations.cake"; // Arguments -var config = CreateFromArguments(); +var config = new BuildConfig +{ + Target = Argument("target", "Default"), + BuildNumber = Argument("buildNumber"), + CurrentRelease = Argument("currentRelease", "1.0.1"), + Branch = Argument("branch"), + BuildPath = Argument("buildPath"), + NuGetApiKey = Argument("nuGetApiKey", ""), + GitHubApiKey = Argument("gitHubApiKey", ""), + Verbose = Argument("verbose", false), + MaxDegreeOfParallelism = Argument("maxDegreeOfParallelism", 5), + CurrentVersion = Argument("currentVersion", "") +}; if (string.IsNullOrEmpty(config.CurrentVersion)) config.CurrentVersion = GetVersion(config); Information($"Current Version: {config.CurrentVersion}"); @@ -80,6 +89,6 @@ Task("Publish") Task("Default") .IsDependentOn("Publish") - .DescriptionFromArguments("Runs the build"); + .Description("Runs the build"); RunTarget(config.Target); \ No newline at end of file diff --git a/build/BuildConfig.cake b/build/BuildConfig.cake index b1de467..550fdd4 100644 --- a/build/BuildConfig.cake +++ b/build/BuildConfig.cake @@ -1,73 +1,22 @@ -using Cake.ArgumentBinder; - public class BuildConfig { - [StringArgument( - "target", - Description = "The target task to execute.", - DefaultValue = "Default")] public string Target { get; set; } - [IntegerArgument( - "buildNumber", - Description = "The current build number.", - Required = true - )] public int BuildNumber { get; set; } - [StringArgument( - "currentRelease", - Description = "The current release version.", - DefaultValue = "1.0.1" - )] public string CurrentRelease { get; set; } - [StringArgument( - "branch", - Description = "The current branch being built.", - Required = true - )] public string Branch { get; set; } - [StringArgument( - "buildPath", - Description = "The base path where the build is being run from.", - Required = true - )] public string BuildPath { get; set; } - [StringArgument( - "nuGetApiKey", - Description = "The API key for interacting with NuGet. Only required when publishing a release.", - DefaultValue = "" - )] public string NuGetApiKey { get; set; } - [StringArgument( - "gitHubApiKey", - Description = "The API key for interacting with GitHub. Only required when publishing a release.", - DefaultValue = "" - )] public string GitHubApiKey { get; set; } - [BooleanArgument( - "verbose", - Description = "Flag to turn on more verbose output.", - DefaultValue = false - )] public bool Verbose { get; set; } - [IntegerArgument( - "maxDegreeOfParallelism", - Description = "Maximum degree of parallelism when running parallel tasks.", - DefaultValue = 5 - )] public int MaxDegreeOfParallelism { get; set; } - [StringArgument( - "currentVersion", - Description = "Override for current version.", - DefaultValue = "" - )] public string CurrentVersion { get; set; } -} \ No newline at end of file +} diff --git a/build/PublishOperations.cake b/build/PublishOperations.cake index 2f57a08..550aba7 100644 --- a/build/PublishOperations.cake +++ b/build/PublishOperations.cake @@ -1,5 +1,5 @@ // Addins -#addin nuget:?package=Cake.Http&version=3.0.2 +#addin nuget:?package=Cake.Http&version=5.1.0 // Usings using System.Text.Json; From 52af9b4134baaa3e63aaf7aef84812c7408d4508 Mon Sep 17 00:00:00 2001 From: Tom Walker Date: Tue, 22 Sep 2026 05:52:02 -0400 Subject: [PATCH 2/2] Remove unnecessary committed review --- agent-context/reviews/copilot-review-2026-09-20.md | 14 -------------- 1 file changed, 14 deletions(-) delete mode 100644 agent-context/reviews/copilot-review-2026-09-20.md diff --git a/agent-context/reviews/copilot-review-2026-09-20.md b/agent-context/reviews/copilot-review-2026-09-20.md deleted file mode 100644 index 55dc6f1..0000000 --- a/agent-context/reviews/copilot-review-2026-09-20.md +++ /dev/null @@ -1,14 +0,0 @@ -# .github/workflows/ci.yml:25-29 - -- On pushes to main, this workflow still runs the default Cake target, which reaches Publish; because this job passes no API keys, DoGitHubRelease/DoNuGetPublish throw and the CI job fails (while the release workflow is running in parallel). Run the Test target for both commands here, or otherwise exclude main from this workflow. - -# agent-context/README.md:1-3 - -- This overview still says CI runs on AppVeyor, but this PR deletes .appveyor.yml and adds GitHub Actions workflows. Agents using this context will be directed to the retired CI system; update the sentence to describe GitHub Actions. -- This issue also appears in the following locations of the same file: - - line 11 - - line 22 - -# agent-context/context/architecture.md:10-13 - -- The architecture snapshot still identifies AppVeyor and .appveyor.yml as the CI implementation, but that file is removed and CI is now in .github/workflows/. Update this current-state row so the architecture documentation does not direct readers to a nonexistent pipeline. \ No newline at end of file