Skip to content

Develop - #35

Merged
ptr727 merged 21 commits into
mainfrom
develop
Apr 5, 2026
Merged

Develop#35
ptr727 merged 21 commits into
mainfrom
develop

Conversation

@ptr727

Copy link
Copy Markdown
Owner

No description provided.

ptr727and others added 18 commits January 15, 2026 11:52
…sion recommendations
Signed-off-by: Pieter Viljoen <ptr727@users.noreply.github.com>
* Branding
Signed-off-by: Pieter Viljoen <ptr727@users.noreply.github.com>
* Allow only squash merging
Signed-off-by: Pieter Viljoen <ptr727@users.noreply.github.com>
---------
Signed-off-by: Pieter Viljoen <ptr727@users.noreply.github.com>
Clarify the requirements for merge rules in the README and ensure all
general merge types are enabled for rulesets to be honored.
---------
Signed-off-by: Pieter Viljoen <ptr727@users.noreply.github.com>
Signed-off-by: Pieter Viljoen <ptr727@users.noreply.github.com>
---------
Signed-off-by: Pieter Viljoen <ptr727@users.noreply.github.com>
Signed-off-by: Pieter Viljoen <ptr727@users.noreply.github.com>
Enhance the README for clearer setup instructions and update branding
for consistency. Adjust merge rules and clarify requirements in the
documentation. Include additional project files and configurations to
support development and code generation tasks.
---------
Signed-off-by: Pieter Viljoen <ptr727@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Improve the logging framework with global configuration options and
refactor the code structure. Enhance documentation across multiple files
and add agent instructions for better clarity.
---------
Signed-off-by: Pieter Viljoen <ptr727@users.noreply.github.com>
Improve code structure and documentation across multiple files, enhance
the logging framework with global configuration options, and update the
testing framework references. Additionally, reorganize the README to
clarify branching workflow requirements.
---------
Signed-off-by: Pieter Viljoen <ptr727@users.noreply.github.com>
Signed-off-by: Pieter Viljoen <ptr727@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Signed-off-by: Pieter Viljoen <ptr727@users.noreply.github.com>
#27)
…improve branch condition checks
Signed-off-by: Pieter Viljoen <ptr727@users.noreply.github.com>
Signed-off-by: Pieter Viljoen <ptr727@users.noreply.github.com>
Signed-off-by: Pieter Viljoen <ptr727@users.noreply.github.com>
Signed-off-by: Pieter Viljoen <ptr727@users.noreply.github.com>
- Bump versions of csharpier, husky, and dotnet-outdated-tool in
dotnet-tools.json
- Require WORKFLOW_PAT secret in run-codegen-pull-request-task.yml for
pull request creation
- Add .Net Outdated Upgrade task in tasks.json for upgrading outdated
packages
- Update Microsoft.Extensions.Http.Resilience, Serilog, and
System.CommandLine versions in CodeGen.csproj, Console.csproj, and
Library.csproj
- Update README.md with instructions for creating and using WORKFLOW_PAT
- Upgrade AwesomeAssertions and Microsoft.NET.Test.Sdk in Tests.csproj
Signed-off-by: Pieter Viljoen <ptr727@users.noreply.github.com>
Signed-off-by: Pieter Viljoen <ptr727@users.noreply.github.com>
Signed-off-by: Pieter Viljoen <ptr727@users.noreply.github.com>
---------
Signed-off-by: Pieter Viljoen <ptr727@users.noreply.github.com>
CopilotAI review requested due to automatic review settings April 5, 2026 21:13

CopilotAI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR updates NuGet/tooling dependencies and improves automation around code generation by having the codegen workflow open labeled PRs using a dedicated PAT, with accompanying documentation and VS Code task updates.

Changes:

  • Bump several NuGet package versions across projects and update local dotnet tool versions.
  • Add a VS Code task for interactive dependency upgrades via dotnet outdated --upgrade:prompt, and document it in project docs.
  • Adjust codegen PR automation to use a reusable-workflow secret (WORKFLOW_PAT) and label-based identification for auto-merge.

Reviewed changes

Copilot reviewed 12 out of 12 changed files in this pull request and generated 1 comment.

Show a summary per file
FileDescription
Tests/Tests.csprojUpdates test dependencies (AwesomeAssertions, Microsoft.NET.Test.Sdk).
Library/Library.csprojUpdates logging abstraction and SourceLink package versions.
Console/Console.csprojUpdates logging abstraction, Serilog, and System.CommandLine versions.
CodeGen/CodeGen.csprojUpdates resilience + CLI/logging package versions for the codegen tool.
.config/dotnet-tools.jsonBumps versions of local dev tools (CSharpier, Husky, dotnet-outdated-tool).
.vscode/tasks.jsonAdds .Net Outdated Upgrade task to run dotnet outdated --upgrade:prompt.
CODESTYLE.mdDocuments the new .Net Outdated Upgrade task.
.github/copilot-instructions.mdDocuments the new .Net Outdated Upgrade task and workflow.
AGENTS.mdUpdates formatting and references the new .Net Outdated Upgrade task.
README.mdDocuments WORKFLOW_PAT and required repo label setup for codegen PR automation.
.github/workflows/run-codegen-pull-request-task.ymlRequires WORKFLOW_PAT, uses it for PR creation, and applies automated-codegen label.
.github/workflows/merge-bot-pull-request.ymlSwitches codegen auto-merge gating from actor-based to label-based detection.

Comment thread.github/workflows/merge-bot-pull-request.yml Outdated
ptr727 added 2 commits April 5, 2026 14:36
…ADME documentation (#36)
Signed-off-by: Pieter Viljoen <ptr727@users.noreply.github.com>
---------
Signed-off-by: Pieter Viljoen <ptr727@users.noreply.github.com>
CopilotAI review requested due to automatic review settings April 5, 2026 22:28

CopilotAI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 7 out of 7 changed files in this pull request and generated 1 comment.

Comment threadREADME.md Outdated
…ADME documentation (#36) (#37)
Signed-off-by: Pieter Viljoen <ptr727@users.noreply.github.com>
---------
Signed-off-by: Pieter Viljoen <ptr727@users.noreply.github.com>
---------
Signed-off-by: Pieter Viljoen <ptr727@users.noreply.github.com>
@ptr727
ptr727 merged commit ea61690 into mainApr 5, 2026
35 checks passed
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@ptr727