Skip to content

Comments

Apply Spotless with new configuration#1567

Merged
gpunto merged 5 commits intodevelopfrom
plugins.spotless.apply
Nov 25, 2025
Merged

Apply Spotless with new configuration#1567
gpunto merged 5 commits intodevelopfrom
plugins.spotless.apply

Conversation

@gpunto
Copy link
Contributor

@gpunto gpunto commented Nov 7, 2025

Goal

As mentioned in #1566, I'm applying spotless in a separate PR

Implementation

Apply spotless

🎨 UI Changes

None

Testing

The check now passes

☑️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 Nov 7, 2025
@github-actions
Copy link
Contributor

github-actions bot commented Nov 7, 2025 •
edited
Loading

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 Nov 7, 2025 •
edited
Loading

SDK Size Comparison 📏

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

@gpunto gpunto force-pushed the plugins.spotless branch 2 times, most recently from 35dabfd to 5841b10 Compare November 11, 2025 11:26
@gpunto gpunto force-pushed the plugins.spotless.apply branch from c3b3fd9 to 35dabfd Compare November 11, 2025 11:29
@gpunto gpunto force-pushed the plugins.spotless.apply branch from 35dabfd to 20685d2 Compare November 14, 2025 12:10
@gpunto gpunto force-pushed the plugins.spotless.apply branch from 20685d2 to 0b166b2 Compare November 25, 2025 12:13
@gpunto gpunto marked this pull request as ready for review November 25, 2025 12:14
@gpunto gpunto requested a review from a team as a code owner November 25, 2025 12:14
@gpunto gpunto marked this pull request as draft November 25, 2025 12:14
Base automatically changed from plugins.spotless to develop November 25, 2025 14:10
@gpunto gpunto marked this pull request as ready for review November 25, 2025 14:10
@sonarqubecloud
Copy link

sonarqubecloud bot commented Nov 25, 2025

Quality Gate Passed 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

@gpunto gpunto merged commit 1fef2b8 into develop Nov 25, 2025
16 of 17 checks passed
@gpunto gpunto deleted the plugins.spotless.apply branch November 25, 2025 14:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

pr:internal Internal or infra-only changes

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@gpunto