Skip to content

docs: implement publishing pipeline (#617)#644

Closed
planetf1 wants to merge 4 commits intogenerative-computing:mainfrom
planetf1:docs/publish-pipeline-617
Closed

docs: implement publishing pipeline (#617)#644
planetf1 wants to merge 4 commits intogenerative-computing:mainfrom
planetf1:docs/publish-pipeline-617

Conversation

@planetf1
Copy link
Contributor

@planetf1 planetf1 commented Mar 13, 2026

docs: implement publishing pipeline

Type of PR

  • Bug Fix
  • New Feature
  • Documentation
  • Other

Description

Fixes #617

Draft — will be updated once PRs #601 and #611 merge.

This PR cannot be fully tested until those two land, as they provide the
static docs content and the API docs build tooling respectively.

Implements Option B from #617: dedicated orphan deployment branches for
Mintlify. CI builds and validates documentation, then pushes the combined
output (static guides + generated API reference) to docs/staging (from
main) or docs/production (from releases).

Changes

  • .github/workflows/docs-publish.yml — new unified workflow replacing
    docs-autogen-pr.yml. Triggers on push to main, release published, PRs
    (validation only), and workflow_dispatch (with force_publish override
    for testing). Validation is soft-fail by default; strict_validation
    input can be enabled when ready.
  • .github/workflows/docs-autogen-pr.yml — deleted (replaced by above).
  • .pre-commit-config.yaml — adds two informational hooks: MDX
    validation when doc files change, docstring quality audit when Python
    source changes. Both skip gracefully if generated docs aren't present.
  • docs/PUBLISHING.md — documents the publishing architecture, branch
    strategy, local development workflow, and Mintlify configuration.
  • .gitignore — adds entries for generated docs/docs/api/ and
    docs/docs/api-reference.mdx.

Dependencies

Requires these PRs to merge first:

Nice to have but not blocking:

Testing

  • Tests added to the respective file if code was changed
  • New code has 100% coverage if code as added
  • Ensure existing tests and github automation passes (a maintainer will kick off the github automation when the rest of the PR is populated)

Testing plan (post-merge of dependencies):

  • Run workflow_dispatch with force_publish=true and a test target branch
  • Verify Mintlify picks up content from the deployment branch
  • Validate markdownlint and coverage audit pass in CI
  • Flip strict_validation once remaining issues are resolved

@github-actions
Copy link
Contributor

The PR description has been updated. Please fill out the template for your PR to be reviewed.

@mergify
Copy link

mergify bot commented Mar 13, 2026

Merge Protections

Your pull request matches the following merge protections and will not be merged until they are valid.

🟢 Enforce conventional commit

Wonderful, this rule succeeded.

Make sure that we follow https://www.conventionalcommits.org/en/v1.0.0/

  • title ~= ^(fix|feat|docs|style|refactor|perf|test|build|ci|chore|revert|release)(?:\(.+\))?:

@planetf1
Copy link
Contributor Author

Not ready for review yet. Opening to share the idea -- we need the mandatory core PRs merged first

@planetf1 planetf1 force-pushed the docs/publish-pipeline-617 branch from 69b3b2e to 30529c0 Compare March 13, 2026 10:47
planetf1 and others added 2 commits March 13, 2026 14:36
Add docs-publish.yml workflow that builds, validates, and deploys
docs to orphan deployment branches (docs/staging from main,
docs/production from releases). Replaces docs-autogen-pr.yml.

Includes pre-commit hooks for MDX validation and docstring quality,
docs/PUBLISHING.md strategy document, and .gitignore entries for
generated API docs.

Validation is soft-fail by default with strict_validation toggle.
workflow_dispatch supports force_publish for testing.
Safer default for manual dispatch — testing deploys to docs/preview
rather than docs/staging. The automatic paths (push→staging,
release→production) are unaffected.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@planetf1 planetf1 force-pushed the docs/publish-pipeline-617 branch from 30529c0 to 11f3d8f Compare March 13, 2026 14:38
planetf1 and others added 2 commits March 13, 2026 14:39
PRs with the 'docs-preview' label deploy to docs/preview branch.
PRs without the label only run build + validation (no deploy).
Also triggers on 'labeled' event so adding the label fires the workflow.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Update PUBLISHING.md with docs/preview branch, label-based PR
deployment, and updated manual dispatch instructions.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@planetf1 planetf1 added the docs-preview Deploy docs to docs/preview branch for pipeline validation label Mar 13, 2026
@planetf1
Copy link
Contributor Author

Recreating PR from upstream branch to enable deploy testing. See replacement PR.

@planetf1
Copy link
Contributor Author

Replaced by #646 (same changes, pushed to upstream branch to enable deploy testing).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docs-preview Deploy docs to docs/preview branch for pipeline validation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

docs: implement publishing pipeline

1 participant