Skip to content

[codex] Add docs PR assignment workflow#771

Merged
jpetey75 merged 2 commits into
mainfrom
codex/assign-docs-pr-authors
Jun 4, 2026
Merged

[codex] Add docs PR assignment workflow#771
jpetey75 merged 2 commits into
mainfrom
codex/assign-docs-pr-authors

Conversation

@jpetey75
Copy link
Copy Markdown
Contributor

@jpetey75 jpetey75 commented Jun 4, 2026

Summary

Adds automation to assign Mintlify docs PRs to the engineer who authored the related lightdash/lightdash source PR.

Changes

  • Add .github/workflows/assign-docs-pr.yml using actions/github-script@v7.
  • Guard the workflow to Mintlify PRs using the confirmed event login mintlify[bot].
  • Parse source PR references from PR bodies, look up the source author, skip bot authors, assign the docs PR, and post cc @author if GitHub silently rejects the assignee.
  • Add scripts/backfill-docs-pr-assignees.sh with DRY_RUN=1 default behavior for one-off or future backfills.

Validation

  • bash -n scripts/backfill-docs-pr-assignees.sh
  • ruby -e 'require "yaml"; YAML.load_file(".github/workflows/assign-docs-pr.yml"); puts "workflow yaml ok"'
  • Ran the backfill against open unassigned PRs after dry-run confirmation. All assignable PRs were assigned, and only PRs with no source owner clue remained unassigned.

@mintlify
Copy link
Copy Markdown
Contributor

mintlify Bot commented Jun 4, 2026

Preview deployment for your docs. Learn more about Mintlify Previews.

Project Status Preview Updated (UTC)
lightdash 🟢 Ready View Preview Jun 4, 2026, 10:46 PM

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Jun 4, 2026

🤖 Documentation Bot

Status

⚠️ 1 broken link needs manual attention


🔗 Broken Links (Manual Fix Required)

Click to view details
   📄 timezones-draft.mdx

- Add a `cc @author` fallback in the live workflow so docs PRs that only
  credit the source author via a cc line (and have no source PR link or
  timeline cross-reference) still get assigned, matching the backfill.
- Wrap the cross-repo pulls.get in try/catch so a deleted or mis-parsed
  source PR number degrades to the cc fallback instead of failing the job.
- Paginate the timeline lookup so cross-references past the first 100
  events are still found.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Jun 4, 2026

🤖 Documentation Bot

Status

⚠️ 1 broken link needs manual attention


🔗 Broken Links (Manual Fix Required)

Click to view details
   📄 timezones-draft.mdx

@jpetey75 jpetey75 marked this pull request as ready for review June 4, 2026 23:09
@jpetey75 jpetey75 merged commit 246ef7a into main Jun 4, 2026
4 of 5 checks passed
@jpetey75 jpetey75 deleted the codex/assign-docs-pr-authors branch June 4, 2026 23:10
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