Skip to content

Revert "CI Test: Run e2e cron tests on Android 16 and 17"#6580

Merged
andremion merged 1 commit into
developfrom
revert-6575-tests/rahullohra/android-17-e2e-cron
Jul 20, 2026
Merged

Revert "CI Test: Run e2e cron tests on Android 16 and 17"#6580
andremion merged 1 commit into
developfrom
revert-6575-tests/rahullohra/android-17-e2e-cron

Conversation

@andremion

@andremion andremion commented Jul 20, 2026

Copy link
Copy Markdown
Contributor

Reverts #6575 until we manage to make that work properly

Summary by CodeRabbit

  • Tests
    • Updated the nightly Android end-to-end test matrix by removing API levels 36 and 37.
    • Existing test coverage for the remaining configured Android API levels is unchanged.

@andremion
andremion requested a review from a team as a code owner July 20, 2026 13:44
@linear

linear Bot commented Jul 20, 2026

Copy link
Copy Markdown

Issue reopened: AND-1315 Chat: Run E2E cron on Android 16 and 17

@github-actions

github-actions Bot commented Jul 20, 2026

Copy link
Copy Markdown
Contributor

PR checklist ❌

The following issues were detected:

  • Section Goal is missing or empty.
  • Section Implementation is missing or empty.
  • Section Testing is missing or empty.
  • Linked issue missing. Add a Linear ticket reference (e.g. AND-123, Closes AND-123, or a https://linear.app/... link) or a GitHub issue (Closes #123) to the PR description.

What we check

  1. Title is concise (5–18 words) unless labeled pr:ignore-for-release.
  2. At least one pr: label exists (e.g., pr:bug, pr:new-feature).
  3. Sections ### Goal, ### Implementation, and ### Testing contain content. Bot-authored PRs are exempt.
  4. PR description references an issue (Linear ticket like AND-123, a Linear URL, or a GitHub Closes #N). Bot-authored PRs are exempt.

@andremion
andremion merged commit 18d80b3 into develop Jul 20, 2026
12 of 13 checks passed
@andremion
andremion deleted the revert-6575-tests/rahullohra/android-17-e2e-cron branch July 20, 2026 13:45
@coderabbitai

coderabbitai Bot commented Jul 20, 2026

Copy link
Copy Markdown

Review Change Stack

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: 059ee610-8f9a-4ab2-b89a-1f279d098e8c

📥 Commits

Reviewing files that changed from the base of the PR and between 0dadef8 and 534b79c.

📒 Files selected for processing (1)
  • .github/workflows/e2e-test-cron.yml

Disabled knowledge base sources:

  • Linear integration is disabled

You can enable these sources in your CodeRabbit configuration.


Walkthrough

The nightly E2E Compose workflow removes Android API levels 37 and 36 from its matrix, leaving the configured levels from 35 through 28 unchanged.

Changes

Nightly Android matrix

Layer / File(s) Summary
Remove API levels 37 and 36
.github/workflows/e2e-test-cron.yml
The nightly run-compose-tests-nightly matrix now starts at Android API level 35 and continues through 28.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Possibly related PRs

Suggested labels: pr:ci

Poem

A bunny checks the nightly queue,
And trims two APIs from view.
Thirty-five to twenty-eight remain,
Hopping smoothly through the lane.
CI munches greens tonight! 🐇

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch revert-6575-tests/rahullohra/android-17-e2e-cron

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@sonarqubecloud

Copy link
Copy Markdown

@github-actions

Copy link
Copy Markdown
Contributor

SDK Size Comparison 📏

SDK Before After Difference Status
stream-chat-android-client 5.95 MB 5.95 MB 0.00 MB 🟢
stream-chat-android-ui-components 11.21 MB 11.21 MB 0.00 MB 🟢
stream-chat-android-compose 12.69 MB 12.69 MB 0.00 MB 🟢

@andremion andremion added the pr:ignore-for-release Exclude from release notes label Jul 21, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr:ignore-for-release Exclude from release notes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant