Skip to content

Merge docs workflows into single release-triggered pipeline#303

Merged
OnestarLee merged 1 commit intomainfrom
feat/merge-docs-workflows
Apr 8, 2026
Merged

Merge docs workflows into single release-triggered pipeline#303
OnestarLee merged 1 commit intomainfrom
feat/merge-docs-workflows

Conversation

@sd-katherinekim
Copy link
Copy Markdown
Contributor

Summary

  • Consolidate notify-docs-update.yml and release-docs-check.yml into a single workflow
  • Trigger: release: [published] + workflow_dispatch (manual re-run)
  • Remove release-docs-check.yml which used pull_request: [closed] and issue_comment: [created] triggers, causing noisy failures on non-release PRs

Changes

Before After
notify-docs-update.yml — sendbird-docs dispatch only notify-docs-update.yml — docs-validation + sendbird-docs dispatch
release-docs-check.yml — docs-validation via PR close/comment Removed (superseded)

Context

Same change as sendbird-uikit-react PR #1415. Both repos had the same duplicate workflow pattern.

Test plan

  • Verify workflow YAML is valid (actionlint passed locally)
  • Verify DOCS_REPO_TOKEN and SDK_REPO_TOKEN org secrets exist
  • On next release: both docs-validation and notify-docs jobs trigger

🤖 Generated with Claude Code

Consolidate notify-docs-update.yml and release-docs-check.yml into a
single workflow triggered by release:published events. The separate
release-docs-check.yml used pull_request:closed and issue_comment
triggers which caused noisy failures on non-release PRs.

Changes:
- Add docs-validation job to notify-docs-update.yml (triggers
  claude-docs-validation on client-workflows)
- Add workflow_dispatch for manual re-runs
- Remove release-docs-check.yml (superseded)

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@codecov-commenter
Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 10.84%. Comparing base (c6e975a) to head (97f562a).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #303   +/-   ##
=======================================
  Coverage   10.84%   10.84%           
=======================================
  Files         360      360           
  Lines        9084     9084           
  Branches     2442     2442           
=======================================
  Hits          985      985           
  Misses       8098     8098           
  Partials        1        1           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@OnestarLee OnestarLee added this pull request to the merge queue Apr 8, 2026
Merged via the queue into main with commit 5a1f209 Apr 8, 2026
8 checks passed
@OnestarLee OnestarLee deleted the feat/merge-docs-workflows branch April 8, 2026 07:07
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.

3 participants