Bump android-ci pin so Sonar analyses develop - #214
aleksandar-apostolov wants to merge 1 commit into
Conversation
Picks up the Sonar step fix from stream-build-conventions-android#78. Until it lands, SonarCloud's develop baseline for this repo is frozen and every PR gate measures new code against it.
PR checklist ✅All required conditions are satisfied:
🎉 Great job! This PR is ready for review. |
SDK Size Comparison 📏
|
|
|
Understand this PR’s impact Explore downstream dependencies and potential security impact with Blast Radius. No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Repository UI Review profile: CHILL Plan: Advanced Run ID: 📒 Files selected for processing (1)
Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review. WalkthroughThe Android CI workflow now references updated pinned conventions workflow commit ChangesAndroid CI
Priority: ⬇️ Low Estimated code review effort: 1 (Trivial) | ~3 minutes Change: Bug fix 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
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. A rabbit checks the workflow pin Comment |



Goal
Refs AND-1443
SonarCloud last analysed this repo's
developon 2025-10-15. The Sonar step in theshared workflow is gated on a
pull_requestfield that is empty on push, so it hasonly ever run on PRs and never refreshes the branch baseline. Every PR gate here
measures new code against that frozen baseline.
This repo went stale before the shared workflow existed (created 2025-12-23), and its
SONAR_TOKENwas only added 2026-04-15 — so the missing token is the likely earliercause, with the push gate keeping it stale since.
Implementation
Bump the
android-ci.ymlpin in.github/workflows/android.ymlto pick upGetStream/stream-build-conventions-android#78. No other pin touched.
Testing
actionlint .github/workflows/android.yml— clean.Draft until #78 merges. It is currently pinned to that PR's branch commit
(
2e7e495). The conventions repo squash-merges and cuts version tags — the previouspin
8b36d37disv0.14.0— so this must be re-pinned to the new release before itcan merge. Expect a large one-off issue count on the first clean
developanalysis;that is the backlog becoming visible, not a regression.
Summary by CodeRabbit