Skip to content

Validate PRs against opensiddur-ai's schema and referential-integrity checks - #15

Merged
EfraimFeinstein merged 1 commit into
mainfrom
feat/validate-prs
Aug 31, 2026
Merged

Validate PRs against opensiddur-ai's schema and referential-integrity checks#15
EfraimFeinstein merged 1 commit into
mainfrom
feat/validate-prs

Conversation

@EfraimFeinstein

Copy link
Copy Markdown
Member

Summary

Adds .github/workflows/validate.yml, a thin caller for opensiddur-ai's new reusable validate-projects.yml workflow (opensiddur-ai#94, opensiddur/opensiddur-ai#98). On every PR here it will:

  1. Schema-validate (RelaxNG + Schematron) the XML of every project the PR touches.
  2. Rebuild the reference database from the PR's own files and check that every URN reference resolves.
  3. Check repo-wide versification agreement across the Tanakh projects (wlc, miqra_al_pi_hamasorah, jps1917).

Runs on pull_request (works for fork PRs, no secrets needed) and is also workflow_dispatch-able with an opensiddur_ai_ref input, to run against a different branch of the validators in opensiddur-ai.

Depends on opensiddur/opensiddur-ai#98 merging to main first — this caller pins uses: opensiddur/opensiddur-ai/.github/workflows/validate-projects.yml@main.

Follow-up (not in this PR, needs repo admin)

Once this workflow has run at least once on a PR, note its exact check-context name (expected Validate projects / validate) and add it as a required status check under Settings → Branches → main, so a failing PR is actually blocked from merging.

🤖 Generated with Claude Code

…ecks (opensiddur-ai#94)

A thin caller for opensiddur-ai's reusable validate-projects workflow:
runs on every pull_request (fork-safe, no secrets needed) and can also
be dispatched manually against a chosen opensiddur-ai branch's
validators via the opensiddur_ai_ref input.

Once this merges, making the check required needs a manual step: add
its status context to main's branch protection (Settings -> Branches).

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
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