Skip to content

CI: gate docs publication on release tags#29

Merged
dknauss merged 4 commits into
mainfrom
docs/artifact-version-and-state-refresh
Jun 17, 2026
Merged

CI: gate docs publication on release tags#29
dknauss merged 4 commits into
mainfrom
docs/artifact-version-and-state-refresh

Conversation

@dknauss

@dknauss dknauss commented Jun 17, 2026

Copy link
Copy Markdown
Owner

Summary

  • add release-metadata validation for the benchmark source and changelog
  • convert generate-docs.yml into a manual preview/build workflow
  • move full PDF/DOCX/EPUB publication into the tag-driven release workflow
  • narrow artifact-validation triggers so ordinary source edits do not require regenerated committed artifacts
  • refresh the historical .planning/STATE.md framing so it no longer reads like active unfinished work

Why

This repo was publishing generated artifacts on ordinary main pushes, which makes version/date drift and artifact churn easy to introduce. This PR moves publication to the release boundary and adds an explicit metadata validator so version/date integrity can be checked separately from full artifact generation.

It also fixes the misleading bootstrap planning state, which still read as if Phase 1 were pending even though the repo has long since shipped.

Validation

  • python3 .github/scripts/validate-release-metadata.py --markdown WordPress-Security-Benchmark.md --changelog CHANGELOG.md
  • bash .github/scripts/verify-metrics.sh docs/current-metrics.md

@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.
To continue using code reviews, you can upgrade your account or add credits to your account and enable them for code reviews in your settings.

@dknauss dknauss merged commit 2349637 into main Jun 17, 2026
2 checks passed
@dknauss dknauss deleted the docs/artifact-version-and-state-refresh branch June 17, 2026 15:56
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