Skip to content

Conversation

@gpunto
Copy link
Contributor

@gpunto gpunto commented Oct 30, 2025

Goal

Part of AND-864

We now have started publishing Stream Gradle plugins, so I'm doing the initial integration here. Atm they don't do a lot (they just configure the Java version), but I'm building upon this first version to move more and more shared configuration to them.

Implementation

  • Replace custom configuration with convention plugins
  • Rename video-specific Gradle plugin to have video. in their coordinate, to avoid clashing with generic ones

🎨 UI Changes

None

Testing

Verify that the project & tests build and run as usual.

☑️Contributor Checklist

General

  • I have signed the Stream CLA (required)
  • Assigned a person / code owner group (required)
  • Thread with the PR link started in a respective Slack channel (required internally)
  • PR targets the develop branch
  • PR is linked to the GitHub issue it resolves

Code & documentation

  • Changelog is updated with client-facing changes
  • New code is covered by unit tests
  • Comparison screenshots added for visual changes
  • Affected documentation updated (KDocs, docusaurus, tutorial)
  • Tutorial starter kit updated
  • Examples/guides starter kits updated (stream-video-examples)

☑️Reviewer Checklist

  • XML sample runs & works
  • Compose sample runs & works
  • Tutorial starter kit
  • Example starter kits work
  • UI Changes correct (before & after images)
  • Bugs validated (bugfixes)
  • New feature tested and works
  • Release notes and docs clearly describe changes
  • All code we touched has new or updated KDocs
  • Check the SDK Size Comparison table in the CI logs

🎉 GIF

Please provide a suitable gif that describes your work on this pull request

@gpunto gpunto added the pr:internal Internal or infra-only changes label Oct 30, 2025
@github-actions
Copy link
Contributor

github-actions bot commented Oct 30, 2025

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.

🎉 Great job! This PR is ready for review.

@github-actions
Copy link
Contributor

github-actions bot commented Oct 30, 2025

SDK Size Comparison 📏

SDK Before After Difference Status
stream-video-android-core 11.91 MB 11.91 MB 0.00 MB 🟢
stream-video-android-ui-xml 5.68 MB 5.68 MB 0.00 MB 🟢
stream-video-android-ui-compose 6.25 MB 6.25 MB 0.00 MB 🟢

@gpunto gpunto marked this pull request as ready for review October 30, 2025 13:47
@gpunto gpunto requested a review from a team as a code owner October 30, 2025 13:47
@gpunto gpunto enabled auto-merge (squash) October 30, 2025 16:17
@sonarqubecloud
Copy link

@gpunto gpunto merged commit ef33f82 into develop Oct 30, 2025
11 of 12 checks passed
@gpunto gpunto deleted the plugins branch October 30, 2025 16:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr:internal Internal or infra-only changes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants