Skip to content

Conversation

@jamesnrokt
Copy link
Collaborator

@jamesnrokt jamesnrokt commented Feb 4, 2026

Background

  • Currently we have many workflows that are triggered by a pull request
  • This PR consolidates them into a single file while allowing them to be called individually with workflow_dispatch, allowing developers to test as needed before opening their PR
  • Finally the notification is now sent only once everything passes instead of just after a simple build job
  • This provides a quick overview of test progress without clicking into many workflows

What Has Changed

  • Github Actions workflows

Screenshots/Video

Screenshot 2026-02-04 at 4 57 59 PM

Checklist

  • I have performed a self-review of my own code.
  • I have made corresponding changes to the documentation.
  • I have added tests that prove my fix is effective or that my feature works.
  • I have tested this locally.

Additional Notes

  • N/A

Reference Issue (For employees only. Ignore if you are an outside contributor)

  • Closes N/A

@jamesnrokt jamesnrokt requested a review from a team as a code owner February 4, 2026 21:52
@github-actions
Copy link

github-actions bot commented Feb 4, 2026

📦 SDK Size Impact Report

Measures how much the SDK adds to an app's size (with-SDK minus without-SDK).

Metric Target Branch This PR Change
App Bundle Impact 1.75 MB 1.75 MB +N/A
Executable Impact 896 bytes 896 bytes +N/A
XCFramework Size 9.38 MB 9.38 MB +N/A

➡️ SDK size impact change is minimal.

Raw measurements

Target branch (main):

{"baseline_app_size_kb":84,"baseline_executable_size_bytes":75464,"with_sdk_app_size_kb":1876,"with_sdk_executable_size_bytes":76360,"sdk_impact_kb":1792,"sdk_executable_impact_bytes":896,"xcframework_size_kb":9604}

This PR:

{"baseline_app_size_kb":84,"baseline_executable_size_bytes":75464,"with_sdk_app_size_kb":1876,"with_sdk_executable_size_bytes":76360,"sdk_impact_kb":1792,"sdk_executable_impact_bytes":896,"xcframework_size_kb":9604}

Copy link
Collaborator

@BrandonStalnaker BrandonStalnaker left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@jamesnrokt jamesnrokt merged commit 37335eb into main Feb 5, 2026
15 checks passed
@jamesnrokt jamesnrokt deleted the ci/consolidate-pull-request-workflows branch February 5, 2026 15:01
denischilik added a commit that referenced this pull request Feb 10, 2026
…ion/9.0-Merge-from-main

* commit '37335eb11c9d858c794b27bbeb91318838dc96c4':
  ci: Consolidate PR workflows (#554)
  fix:  race condition in forwardQueue causing crash during SDK init (#553)
  chore: Update Example Apps (#555)
  chore: Prevent Version changes from breaking integration tests (#548)

# Conflicts:
#	.github/workflows/build-secondary-platforms.yml
BrandonStalnaker pushed a commit that referenced this pull request Feb 11, 2026
* ci: Consolidate PR workflows

* Fix concurrency

* Handle modulemap issue

* Update cross-platform-tests.yml

* Downgrade xcode for secondary platform tests

Kotlin 1.9.0 is incompatible with Xcode 16.x SDKs. Use macOS-14 with Xcode 15.4.

macOS-14 deprecated Nov 2025; Kotlin upgrade needed before then.

https://youtrack.jetbrains.com/issue/KT-69093
actions/runner-images#13518

* Revert changes and change head_ref usage

* Revert ross-platform-tests

(cherry picked from commit 37335eb)
BrandonStalnaker pushed a commit that referenced this pull request Feb 12, 2026
* ci: Consolidate PR workflows

* Fix concurrency

* Handle modulemap issue

* Update cross-platform-tests.yml

* Downgrade xcode for secondary platform tests

Kotlin 1.9.0 is incompatible with Xcode 16.x SDKs. Use macOS-14 with Xcode 15.4.

macOS-14 deprecated Nov 2025; Kotlin upgrade needed before then.

https://youtrack.jetbrains.com/issue/KT-69093
actions/runner-images#13518

* Revert changes and change head_ref usage

* Revert ross-platform-tests

(cherry picked from commit 37335eb)
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