Skip to content

Merge develop to main: template re-sync - #186

Merged
ptr727 merged 1 commit into
mainfrom
develop
Jun 23, 2026
Merged

Merge develop to main: template re-sync#186
ptr727 merged 1 commit into
mainfrom
develop

Conversation

@ptr727

@ptr727 ptr727 commented Jun 23, 2026

Copy link
Copy Markdown
Owner

Promote the template re-sync from develop (#185) to main.

  • AGENTS.md: PR Review Etiquette Merge Gate + codegen-determinism rule + review-loop wording refinements.
  • publish-release.yml: fail-fast guard for workflow_dispatch off the default branch.
  • .editorconfig / .markdownlint-cli2.jsonc: comment-wording fixes.
  • CRLF normalization for dependabot.yml and the workflow .yml files.

Maintenance-only promotion: no functional library change, version.json unchanged (NBGV advances main's height).

…185)

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.
Copilot AI review requested due to automatic review settings June 23, 2026 22:54

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

Promotes the previously-merged template re-sync from develop to main, updating repo governance/docs and tightening CI/release workflow safety without changing library behavior or versioning.

Changes:

  • Update AGENTS.md with the “Merge Gate” contract language and a deterministic-codegen rule.
  • Add a fail-fast guard in publish-release.yml to prevent workflow_dispatch runs from a non-default ref (avoids mis-versioning in the branch matrix).
  • Minor config comment wording tweaks plus CRLF normalization for dependabot.yml and workflow .yml 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 + codegen determinism guidance and refines review-loop wording.
.markdownlint-cli2.jsonc Clarifies why MD060 is disabled (table pipe spacing style).
.editorconfig Comment wording update (“JSON and JSONC files”).
.github/workflows/publish-release.yml Adds dispatch-on-default-branch guard to prevent mis-versioned publishes.
.github/workflows/test-pull-request.yml CRLF normalization only (no behavioral changes).
.github/workflows/run-periodic-codegen-pull-request.yml CRLF normalization only (no behavioral changes).
.github/workflows/run-codegen-pull-request-task.yml CRLF normalization only (no behavioral changes).
.github/workflows/merge-bot-pull-request.yml CRLF normalization only (no behavioral changes).
.github/workflows/get-version-task.yml CRLF normalization only (no behavioral changes).
.github/workflows/build-release-task.yml CRLF normalization only (no behavioral changes).
.github/workflows/build-nugetlibrary-task.yml CRLF normalization only (no behavioral changes).
.github/workflows/build-datebadge-task.yml CRLF normalization only (no behavioral changes).
.github/dependabot.yml CRLF normalization only (no behavioral changes).

@ptr727
ptr727 merged commit b3e7868 into main Jun 23, 2026
12 checks passed
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