Skip to content

NA-0758: the required-check count is three, not one; NOTICE takes the house form - #45

Merged
Tebbens4832 merged 1 commit into
mainfrom
na0758-required-checks-and-notice
Aug 25, 2026
Merged

NA-0758: the required-check count is three, not one; NOTICE takes the house form#45
Tebbens4832 merged 1 commit into
mainfrom
na0758-required-checks-and-notice

Conversation

@Tebbens4832

Copy link
Copy Markdown
Member

What this changes

Three documentation defects found by the NA-0758 public-facing documentation audit.

1. The required-check count, stated twice and wrong both times (census rows AC-01, AR-01)

CONTRIBUTING.md:9
-3) Every PR must pass the single required `rust` CI check.
+3) Every PR must pass the three required CI checks: `rust`, `advisories`, and `infra-literal-scan`.

README.md:34
-- `main` currently requires only the `rust` check
+- `main` currently requires three checks: `rust`, `advisories`, and `infra-literal-scan`

Measured, at the edit rather than carried from the audit:

gh api repos/QuantumShieldLabs/qsl-attachments/branches/main/protection \
  --jq '.required_status_checks.contexts'
["rust","advisories","infra-literal-scan"]

2. NOTICE takes the house form (census rows AN-01 / LI-05)

This was the one NOTICE of four that did not share the shape the others use. The form it adopts here was verified against qsl-protocol, qsl-server and qsl-desktop at their current mains, not assumed.

⚠️ This is a form change, not a licence change. The repository remains AGPL-3.0-only and the sentence saying so is preserved. No LICENSE byte is touched — the licence text itself is a separate operator-owned track and deliberately does not ride a docs PR.

Scope

Documentation only — three files. No code, test, workflow, licence or manifest byte.

infra-literal-scan --mode diff returns clean (diff; 3 files, 13 lines examined), matching the commit's insertion count exactly.

Provenance

Census rows AC-01, AR-01, AN-01/LI-05 of the NA-0758 audit (89 claims across 48 files in 5 repositories; 56 measured TRUE). The identical required-check defect was measured in qsl-server (PR #70) and qsl-desktop, each fixed in its own PR.

…e house form

Two claims in this repository's public docs said `main` requires a single
`rust` check. Live branch protection requires three: `rust`, `advisories`,
and `infra-literal-scan`.

  CONTRIBUTING.md:9  census row AC-01
  README.md:34       census row AR-01

Measured with
`gh api repos/QuantumShieldLabs/qsl-attachments/branches/main/protection`
on 2026-08-25, re-measured at the edit rather than inherited.

NOTICE (census rows AN-01 / LI-05) is brought into the form the other three
public NOTICE files already share: product-name title line, `Copyright (c)
2026 QuantumShield Labs`, the one-line licence sentence, a canonical
repositories list, and the proof-runs/unaudited-binaries closing. The form
was verified against qsl-protocol, qsl-server and qsl-desktop at their
current mains, not assumed.

This is a FORM change, not a licence change: the repository remains
AGPL-3.0-only and the sentence saying so is preserved. No LICENSE byte is
touched here -- the licence text itself is a separate operator track.

Docs only: three files. No code, test, workflow, licence or manifest byte.
@Tebbens4832
Tebbens4832 merged commit 765659a into main Aug 25, 2026
4 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.

1 participant