Skip to content

Re-sync from template: add Merge Gate, codegen rules, dispatch guard - #185

Merged
ptr727 merged 1 commit into
developfrom
resync-merge-gate
Jun 23, 2026
Merged

Re-sync from template: add Merge Gate, codegen rules, dispatch guard#185
ptr727 merged 1 commit into
developfrom
resync-merge-gate

Conversation

@ptr727

@ptr727 ptr727 commented Jun 23, 2026

Copy link
Copy Markdown
Owner

Re-sync from the upstream ProjectTemplate.

Content changes

  • AGENTS.md - add the PR Review Etiquette Merge Gate section and the "Mandatory in every derived repo" note; add the codegen-determinism rule to the Branching Model; refine the review-loop wording (explicit requestReviews re-request); judgement->judgment.
  • publish-release.yml - fail fast when workflow_dispatch runs off the default branch, since the matrix builds both branches and a non-default ref mis-versions the main leg.
  • .editorconfig / .markdownlint-cli2.jsonc - comment-wording fixes.

EOL

  • Normalizes dependabot.yml and the workflow .yml files to CRLF per .editorconfig; no content change in those files.

Adds the PR Review Etiquette Merge Gate and codegen-determinism rule to
AGENTS.md, a fail-fast guard in publish-release.yml for dispatches off
the default branch, and normalizes workflow files to CRLF per
.editorconfig.
Copilot AI review requested due to automatic review settings June 23, 2026 22:50

Copilot AI 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

Re-syncs LanguageTags from the upstream ProjectTemplate by updating the agent/workflow runbook documentation, tightening release-workflow safety for manual dispatches, and applying minor configuration comment cleanups (plus CRLF normalization for YAML files).

Changes:

  • Extend AGENTS.md with a “Merge Gate” contract and an explicit codegen-determinism rule, and refine review-loop wording/spelling.
  • Add a fail-fast guard in publish-release.yml to prevent workflow_dispatch runs from non-default refs that could mis-version the branch matrix.
  • Minor config comment tweaks and CRLF normalization in YAML/config files.

Reviewed changes

Copilot reviewed 4 out of 13 changed files in this pull request and generated no comments.

Show a summary per file
File Description
AGENTS.md Adds Merge Gate guidance, clarifies review-loop mechanics, and documents deterministic codegen expectations.
.markdownlint-cli2.jsonc Updates commentary for MD060 handling (no behavior change).
.github/workflows/test-pull-request.yml Line-ending normalization only (no logical changes in shown diff).
.github/workflows/run-periodic-codegen-pull-request.yml Line-ending normalization only (no logical changes in shown diff).
.github/workflows/run-codegen-pull-request-task.yml Line-ending normalization only (no logical changes in shown diff).
.github/workflows/publish-release.yml Adds a dispatch guard to ensure manual publishes run from the default branch.
.github/workflows/merge-bot-pull-request.yml Line-ending normalization only (no logical changes in shown diff).
.github/workflows/get-version-task.yml Line-ending normalization only (keeps documented dotnet/nbgv@master exception).
.github/workflows/build-release-task.yml Line-ending normalization only (no logical changes in shown diff).
.github/workflows/build-nugetlibrary-task.yml Line-ending normalization only (no logical changes in shown diff).
.github/workflows/build-datebadge-task.yml Line-ending normalization only (no logical changes in shown diff).
.github/dependabot.yml Line-ending normalization only; retains dual-target main/develop update strategy.
.editorconfig Comment wording tweak (“JSON and JSONC”).

@ptr727
ptr727 merged commit 173d7a0 into develop Jun 23, 2026
8 checks passed
@ptr727
ptr727 deleted the resync-merge-gate branch June 23, 2026 22:53
Sign up for free to 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