Skip to content

[puppetsync] Add a REFERENCE.md freshness check to PR tests - #385

Open
silug wants to merge 1 commit into
simp:masterfrom
silug:puppetsync/20260811-reference-md
Open

[puppetsync] Add a REFERENCE.md freshness check to PR tests#385
silug wants to merge 1 commit into
simp:masterfrom
silug:puppetsync/20260811-reference-md

Conversation

@silug

@silug silug commented Aug 11, 2026

Copy link
Copy Markdown
Contributor

This patch updates pr_tests.yml from the reconciled puppetsync
template, whose reference job now fails when REFERENCE.md is out
of sync with the module's strings docs (rake validate:strings),
and regenerates REFERENCE.md so the new check starts green.

Repo-specific jobs.acceptance blocks are preserved as-is.

@silug
silug force-pushed the puppetsync/20260811-reference-md branch from a8038c0 to 52adfd7 Compare August 11, 2026 19:53
@op-ct op-ct added this to Org Triage Aug 11, 2026
silug added a commit to hcaballero2/puppetsync that referenced this pull request Aug 11, 2026
The repo-specific override (needed for its per-directory spec matrix and
extra acceptance suites) was still the pre-OpenVox 2023 template, so the
fleet run replaced the repo's modern workflow with it — reverting the
OpenVox spec matrix and dropping the reference job it was supposed to
add (caught validating simp/pupmod-simp-simp#385). The override is now
the repo's current master workflow plus the reference job: exactly the
change every other repo received.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
This patch updates pr_tests.yml from the reconciled puppetsync
template, whose `reference` job now fails when REFERENCE.md is out
of sync with the module's strings docs (`rake validate:strings`),
and regenerates REFERENCE.md so the new check starts green.

Repo-specific `jobs.acceptance` blocks are preserved as-is.
@silug
silug force-pushed the puppetsync/20260811-reference-md branch from 52adfd7 to 6adfd17 Compare August 11, 2026 23:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: New

Development

Successfully merging this pull request may close these issues.

2 participants