Skip to content

Require PR approval before release tagging#1693

Merged
ajpallares merged 2 commits into
mainfrom
pallares/require-release-pr-approval
Mar 23, 2026
Merged

Require PR approval before release tagging#1693
ajpallares merged 2 commits into
mainfrom
pallares/require-release-pr-approval

Conversation

@ajpallares

@ajpallares ajpallares commented Mar 20, 2026

Copy link
Copy Markdown
Member

Motivation

The release workflow currently allows tagging a release branch without verifying that the PR has been reviewed and approved. This adds a safeguard to ensure the release PR is properly approved before the tag is created.

Description

Bumps revenuecat/sdks-common-config orb to @3.16.0, which includes a validate_pr_approved step in the tag-current-branch job. This verifies the release PR has been approved by an org member with write permissions before the git tag is created.

Depends on: RevenueCat/sdks-circleci-orb#41


Note

Medium Risk
Changes the release/tagging workflow behavior via an orb upgrade, which could block or alter release tagging if the new PR-approval validation misbehaves or is misconfigured.

Overview
Updates the CircleCI revenuecat/sdks-common-config orb from 3.13.0 to 3.16.0, pulling in new behavior for the revenuecat/tag-current-branch job (including PR-approval validation) before creating release tags.

Written by Cursor Bugbot for commit 604173c. This will update automatically on new commits. Configure here.

This version includes a `validate_pr_approved` check in `tag-current-branch`
that verifies the release PR has been approved before the tag is created.

Made-with: Cursor
@ajpallares ajpallares force-pushed the pallares/require-release-pr-approval branch from 7903031 to b7a623a Compare March 20, 2026 17:41
@ajpallares ajpallares marked this pull request as ready for review March 23, 2026 11:24
@ajpallares ajpallares requested a review from a team as a code owner March 23, 2026 11:24
@ajpallares ajpallares merged commit 6df9c9a into main Mar 23, 2026
14 checks passed
@ajpallares ajpallares deleted the pallares/require-release-pr-approval branch March 23, 2026 11:44
RCGitBot added a commit that referenced this pull request Apr 1, 2026
**This is an automatic release.**

## RevenueCat SDK
### ✨ New Features
* Lower minimum Flutter SDK to 3.22.0 and Dart to 3.4.0 (#1704) via Rick
(@rickvdl)
### 📦 Dependency Updates
* [AUTOMATIC BUMP] Updates purchases-hybrid-common to 17.54.0 (#1703)
via RevenueCat Git Bot (@RCGitBot)
* [Android
9.28.1](https://github.com/RevenueCat/purchases-android/releases/tag/9.28.1)
* [Android
9.28.0](https://github.com/RevenueCat/purchases-android/releases/tag/9.28.0)
* [iOS
5.67.0](https://github.com/RevenueCat/purchases-ios/releases/tag/5.67.0)

### 🔄 Other Changes
* Bump fastlane-plugin-revenuecat_internal from `9a6911b` to `f11fe40`
(#1701) via dependabot[bot] (@dependabot[bot])
* security: pin GitHub Actions to SHA hashes (#1697) via Alfonso
Embid-Desmet (@alfondotnet)
* Bump activesupport from 7.2.3 to 7.2.3.1 (#1696) via dependabot[bot]
(@dependabot[bot])
* Merge release PR after deploy (#1694) via Antonio Pallares
(@ajpallares)
* Require PR approval before release tagging (#1693) via Antonio
Pallares (@ajpallares)

<!-- CURSOR_SUMMARY -->
---

> [!NOTE]
> **Low Risk**
> Primarily a coordinated version/release bump across Flutter, Android,
iOS/macOS, and web with updated dependency versions and minimum SDK
constraints; no functional logic changes beyond version metadata.
> 
> **Overview**
> Bumps the Flutter SDK release from `9.15.1` to `9.16.0` across package
metadata (`.version`, `pubspec.yaml`, podspecs, Gradle files) and
updates the embedded plugin version constants for Android/iOS/web.
> 
> Updates release documentation (`CHANGELOG-LATEST.md`, `CHANGELOG.md`,
`VERSIONS.md`) to include the `9.16.0` notes, including the lowered
minimum Flutter/Dart requirements and the `purchases-hybrid-common` bump
to `17.54.0` (with corresponding native iOS/Android version references).
Also updates `purchases_ui_flutter` to depend on `purchases_flutter
^9.16.0`.
> 
> <sup>Written by [Cursor
Bugbot](https://cursor.com/dashboard?tab=bugbot) for commit
c93cfb2. This will update automatically
on new commits. Configure
[here](https://cursor.com/dashboard?tab=bugbot).</sup>
<!-- /CURSOR_SUMMARY -->
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants