Skip to content
Merged
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
16 changes: 8 additions & 8 deletions WORKFLOW.md
Original file line number Diff line number Diff line change
Expand Up @@ -587,14 +587,14 @@ determined by NBGV from the checkout state in section 3.*
### 5D. Configuration audit

Follow the read-only audit in [`AUDIT.md`](./AUDIT.md) (section 6). It confirms the listed secrets exist,
the `main`/`develop` rulesets enforce the required merge method + status check + signed commits +
strict-off, and the repository settings (auto-merge, allowed merge methods) are in place, reporting any
drift. A missing or incorrect configuration item is a defect (D10). Secret *values* cannot be read
back, so the audit checks that the names exist, reporting a missing name or a failed query as a defect; the
GitHub App installation is a best-effort check (a precise check needs app-level auth, so it notes rather
than reports a defect). The NuGet.org trusted-publishing
policy (D4.7) lives outside GitHub and cannot be checked by `gh api`, so it sits outside AUDIT.md's drift
report entirely - it is a manual checklist item, verified on NuGet.org (see section 6).
the `main`/`develop` rulesets enforce the required merge method + status check + signed commits, with
"require branches up to date before merging" **off**, and the repository settings (auto-merge, allowed
merge methods) are in place, reporting any drift. A missing or incorrect configuration item is a defect
(D10). Secret *values* cannot be read back, so the audit checks that the names exist, reporting a missing
name or a failed query as a defect; the GitHub App installation is a best-effort check (a precise check
needs app-level auth, so it notes rather than reports a defect). The NuGet.org trusted-publishing policy
(D4.7) lives outside GitHub and cannot be checked by `gh api`, so it sits outside AUDIT.md's drift report
entirely - it is a manual checklist item, verified on NuGet.org (see section 6).

### Assessment

Expand Down