Skip to content

refactor: delegate KMP bumps to shared workflow - #2721

Merged
fadi-george merged 2 commits into
mainfrom
fadi/fix-kmp-bump-pr-reopen
Aug 19, 2026
Merged

refactor: delegate KMP bumps to shared workflow#2721
fadi-george merged 2 commits into
mainfrom
fadi/fix-kmp-bump-pr-reopen

Conversation

@fadi-george

Copy link
Copy Markdown
Contributor

Description

One Line Summary

Delegate KMP submodule bump releases to the shared SDK workflow.

Details

Motivation

Remove duplicated release automation while retaining the Android workflow entry point and closed-PR recovery behavior.

Scope

Only the KMP bump workflow changes. Runtime SDK behavior and public APIs are unaffected.

Dependency

Merge sdk-shared PR #55 first because this workflow references the shared workflow on main.

Testing

Unit testing

No unit tests are needed for this workflow-only change. All involved workflows pass actionlint; shared PR-state behavior was tested with mocked open, closed, merged, and missing PRs.

Manual testing

Device testing is not applicable to release automation.

Affected code checklist

  • Notifications
  • Outcomes
  • Sessions
  • In-App Messaging
  • REST API requests
  • Public API changes

Checklist

Overview

  • I have filled out all required sections above
  • PR does one thing
  • Any Public API changes are explained and conform to existing APIs

Testing

  • I have included test coverage or explained why it is not needed
  • All applicable automated tests pass
  • Device testing is not applicable and is explained above

Final pass

  • Code is as readable as possible
  • I have reviewed this PR myself

Made with Cursor

Keep host release entry points while centralizing bump and PR recovery behavior.

Co-authored-by: Cursor <cursoragent@cursor.com>
@fadi-george
fadi-george requested a review from a team as a code owner August 19, 2026 20:46
Comment thread .github/workflows/bump-kmp-submodule.yml Fixed
@github-actions

github-actions Bot commented Aug 19, 2026

Copy link
Copy Markdown
Contributor

📊 Diff Coverage Report

✓ Coverage check passed (no source files changed)

📥 View workflow run

Retain the least privileges required by the delegated workflow and its token fallback.

Co-authored-by: Cursor <cursoragent@cursor.com>
@fadi-george
fadi-george merged commit 8092659 into main Aug 19, 2026
7 checks passed
@fadi-george
fadi-george deleted the fadi/fix-kmp-bump-pr-reopen branch August 19, 2026 21:03
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