Publish API docs on push to main by aleksandar-apostolov · Pull Request #1862 · GetStream/stream-video-android

aleksandar-apostolov · 2026-10-07T12:06:39Z

Goal

Refs AND-1617 — move API-docs publishing to a push-to-main workflow, ahead of dropping it from the release job.

Implementation

New publish-api-docs.yml calls the shared workflow at build-conventions v1.0.1 (d275481) on push to main (+ workflow_dispatch, main only) and passes SLACK_WEBHOOK_ANDROID_CICD for failure alerts.

release.yml stays pinned at 9ea4d20: bumping it pulls in the Stream-repo publishing cutover, which goes in its own PR. Until then the release job still runs Dokka, so docs publish twice per release (same content). The release-time saving lands with that bump.

Testing

actionlint clean. Build-conventions v1.0.1 released through the same path: API Docs ran on the main push and redeployed Pages. After the next release here, it updates getstream.github.io/stream-video-android.

Adds a caller for the shared publish-api-docs workflow so docs no longer depend on the release job. The release.yml pin is unchanged; until it is bumped, docs are published by both.

github-actions · 2026-10-07T12:06:54Z

PR checklist ✅

All required conditions are satisfied:

Title length is OK (or ignored by label).
At least one pr: label exists.
Sections ### Goal, ### Implementation, and ### Testing are filled, or the PR is bot-authored.
An issue is linked (Linear ticket or GitHub issue), or the PR is bot-authored.

🎉 Great job! This PR is ready for review.

github-actions · 2026-10-07T12:10:41Z

SDK Size Comparison 📏

SDK	Before	After	Status
stream-video-android-core	12.38 MB	12.38 MB	🟢
stream-video-android-ui-xml	5.66 MB	5.66 MB	🟢
stream-video-android-ui-compose	6.19 MB	6.19 MB	🟢

coderabbitai · 2026-10-07T12:12:02Z

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Repository UI
Review profile: CHILL
Plan: Advanced
Run ID: 23765d05-a6a1-48bf-bfab-a5ac7caf8ea3

📥 Commits

Reviewing files that changed from the base of the PR and between 622f636 and fc77066.

📒 Files selected for processing (1)

.github/workflows/publish-api-docs.yml

Included review availability: This review used your included allowance. Your plan provides up to 2 included reviews per hour; 1 remain after this review.

Walkthrough

Adds a GitHub Actions workflow that runs on pushes to main or manual dispatch. It calls a pinned reusable workflow with contents: write permission.

Changes

API docs publishing

Layer / File(s)	Summary
Configure API docs publishing `.github/workflows/publish-api-docs.yml`	Adds triggers for pushes to `main` and manual dispatch. The workflow delegates to a pinned reusable workflow and grants it `contents: write` permission.

Priority: ⬇️ Low

Estimated code review effort: 2 (Simple) | ~5 minutes

Change: Feature

Suggested reviewers: andremion

Merge Risk: ⚪ Minimal · up to fc770

This PR adds a workflow that publishes API docs on pushes to main. No concrete defect was found. Before merging, confirm that the reusable-workflow pin points to the merged upstream commit, as the PR description says.

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Title check	✅ Passed	The title clearly describes the main change: publishing API docs when changes are pushed to `main`.
Description check	✅ Passed	The description includes the goal, implementation, and testing details. UI screenshots and checklist items are not relevant to this workflow change.

✨ Finishing Touches

🧪 Generate unit tests (beta)

Commit to this branch
Create a new PR

Autopilot · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

_{A rabbit checks the workflow's cue,
It hops when main has work to do.
A manual tap can start the run,
The docs publish when steps are done.
Fresh pages greet the morning light.}

_{Comment @coderabbitai help to get the list of available commands.}

Re-pins to the conventions commit that alerts on docs failures.

rahul-lohra

lgtm

…pi-docs-publishing-out-of-the-release-workflow

sonarqubecloud · 2026-10-07T13:32:35Z

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

stream-public-bot · 2026-10-07T15:19:26Z

🚀 Available in v1.36.0

ci: publish API docs on push to main

fc77066

Adds a caller for the shared publish-api-docs workflow so docs no longer depend on the release job. The release.yml pin is unchanged; until it is bumped, docs are published by both.

aleksandar-apostolov added the pr:ci CI / GitHub Actions / workflow updates label Oct 7, 2026

aleksandar-apostolov requested a review from a team as a code owner October 7, 2026 12:06

aleksandar-apostolov added the pr:ci CI / GitHub Actions / workflow updates label Oct 7, 2026

aleksandar-apostolov added 2 commits October 7, 2026 14:28

ci: pass the Slack webhook to the API docs workflow

0a9b827

Re-pins to the conventions commit that alerts on docs failures.

ci: pin build-conventions v1.0.1

ec44086

rahul-lohra approved these changes Oct 7, 2026

View reviewed changes

PratimMallick approved these changes Oct 7, 2026

View reviewed changes

Merge branch 'develop' into aleksandarapostolov/and-1617-move-dokka-a…

9a2f685

…pi-docs-publishing-out-of-the-release-workflow

PratimMallick enabled auto-merge October 7, 2026 13:26

PratimMallick merged commit ea955ec into develop Oct 7, 2026
17 checks passed

PratimMallick deleted the aleksandarapostolov/and-1617-move-dokka-api-docs-publishing-out-of-the-release-workflow branch October 7, 2026 13:32

stream-public-bot added the released Included in a release label Oct 7, 2026

Navigation Menu

Repository navigation

Publish API docs on push to main - #1862

Publish API docs on push to main#1862
PratimMallick merged 4 commits into
developfrom
aleksandarapostolov/and-1617-move-dokka-api-docs-publishing-out-of-the-release-workflow

aleksandar-apostolov commented Oct 7, 2026 •

edited

Loading

Uh oh!

github-actions Bot commented Oct 7, 2026

Uh oh!

github-actions Bot commented Oct 7, 2026 •

edited

Loading

Uh oh!

coderabbitai Bot commented Oct 7, 2026 •

edited

Loading

Uh oh!

rahul-lohra left a comment

Uh oh!

sonarqubecloud Bot commented Oct 7, 2026

Uh oh!

Uh oh!

stream-public-bot commented Oct 7, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

Conversation

aleksandar-apostolov commented Oct 7, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Goal

Implementation

Testing

Uh oh!

github-actions Bot commented Oct 7, 2026

PR checklist ✅

Uh oh!

github-actions Bot commented Oct 7, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

SDK Size Comparison 📏

Uh oh!

coderabbitai Bot commented Oct 7, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Walkthrough

Changes

Uh oh!

rahul-lohra left a comment

Choose a reason for hiding this comment

Uh oh!

sonarqubecloud Bot commented Oct 7, 2026

Quality Gate passed

Uh oh!

Uh oh!

stream-public-bot commented Oct 7, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

aleksandar-apostolov commented Oct 7, 2026 •

edited

Loading

github-actions Bot commented Oct 7, 2026 •

edited

Loading

coderabbitai Bot commented Oct 7, 2026 •

edited

Loading