Skip to content

Conversation

@gpunto
Copy link
Collaborator

@gpunto gpunto commented Dec 23, 2025

Goal

AND-961: Move shared workflows to conventions repo

We're centralizing the workflows with the plugins in the build conventions repo, so we're changing the references to point to them

Implementation

Change the workflow references

Testing

The workflows should run

Checklist

  • Issue linked (if any)
  • Tests/docs updated
  • I have signed the Stream CLA (required for external contributors)

Summary by CodeRabbit

  • Chores
    • Updated internal CI/CD workflow configurations and references to newer versions for improved build consistency and reliability.

✏️ Tip: You can customize this high-level summary in your review settings.

@gpunto gpunto added the pr:ci CI changes label Dec 23, 2025
@gpunto gpunto force-pushed the and-961/move-workflows branch from ecc122f to cb5d50f Compare December 23, 2025 16:42
@github-actions
Copy link
Contributor

github-actions bot commented Dec 23, 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.

@gpunto gpunto force-pushed the and-961/move-workflows branch from cb5d50f to b80d56e Compare December 23, 2025 16:44
@sonarqubecloud
Copy link

@gpunto gpunto marked this pull request as ready for review December 24, 2025 08:24
@aleksandar-apostolov aleksandar-apostolov merged commit 8c9e16c into develop Dec 24, 2025
6 checks passed
@aleksandar-apostolov aleksandar-apostolov deleted the and-961/move-workflows branch December 24, 2025 08:26
@coderabbitai
Copy link

coderabbitai bot commented Dec 24, 2025

Caution

Review failed

The pull request is closed.

Walkthrough

GitHub workflow configuration files updated to reference shared workflows from the new stream-build-conventions-android repository at version v0.6.0 instead of the old android-ci-actions repository. Permissions block added to one workflow for proper access control.

Changes

Cohort / File(s) Summary
Workflow reference migration
.github/workflows/pr-clean-stale.yaml, .github/workflows/pr-quality.yml
Updated workflow references from GetStream/android-ci-actions to GetStream/[email protected]; added permissions block to pr-clean-stale.yaml granting write access to contents, pull-requests, and issues
Code cleanup & formatting
.github/workflows/pr-quality.yml
Removed caller-comment line and standardized pull_request types array formatting with consistent spacing

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

Suggested reviewers

  • VelikovPetar
  • aleksandar-apostolov

Poem

🐰 Workflows hop to a new warren,
Shared conventions now in one place,
Permissions granted with care,
Cleaner paths, better embrace,
Build magic flows everywhere!

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch and-961/move-workflows

📜 Recent review details

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Disabled knowledge base sources:

  • Linear integration is disabled by default for public repositories

You can enable these sources in your CodeRabbit configuration.

📥 Commits

Reviewing files that changed from the base of the PR and between 0a68ffb and b80d56e.

📒 Files selected for processing (2)
  • .github/workflows/pr-clean-stale.yaml
  • .github/workflows/pr-quality.yml

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 and usage tips.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr:ci CI changes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants