Skip to content

Require PR approval before release tagging#3268

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

Require PR approval before release tagging#3268
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 by pulling in a newer CircleCI orb; misconfiguration or orb regressions could block or delay release tagging.

Overview
Updates the CircleCI revenuecat/sdks-common-config orb from 3.13.0 to 3.16.0.

This is intended to add a safeguard to release tagging (via the orb’s tag-current-branch job) by validating that the associated release PR has been approved before creating git tags.

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

@codecov

codecov Bot commented Mar 20, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 79.38%. Comparing base (bb538ed) to head (1d68cd9).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #3268   +/-   ##
=======================================
  Coverage   79.38%   79.38%           
=======================================
  Files         357      357           
  Lines       14347    14347           
  Branches     1959     1959           
=======================================
  Hits        11389    11389           
  Misses       2154     2154           
  Partials      804      804           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

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 974ff98 to fb850a1 Compare March 20, 2026 17:42
@ajpallares ajpallares marked this pull request as ready for review March 23, 2026 11:23
@ajpallares ajpallares requested a review from a team as a code owner March 23, 2026 11:23
@ajpallares ajpallares added this pull request to the merge queue Mar 23, 2026
Merged via the queue into main with commit 06f1122 Mar 23, 2026
35 checks passed
@ajpallares ajpallares deleted the pallares/require-release-pr-approval branch March 23, 2026 12:17
github-merge-queue Bot pushed a commit that referenced this pull request Mar 26, 2026
**This is an automatic release.**

## RevenueCat SDK
### 🐞 Bugfixes
* [EXTERNAL] fix: ensure activity is attached before showing in-app
messages (#3274) contributed by @matteinn (#3275) via Toni Rico
(@tonidero)
* Ensure MediaPlayer has dedicated thread owner that is not the main
thread (#3148) via Jacob Rakidzich (@JZDesign)
* Fix heartbeat monitor and Slack notifications for nightly integration
tests (#3259) via Rick (@rickvdl)

## RevenueCatUI SDK
### Paywallv2
#### ✨ New Features
* Feat: Restore gating in paywalls UI (#3171) via Jacob Rakidzich
(@JZDesign)

### 🔄 Other Changes
* security: pin GitHub Actions to SHA hashes (#3272) via Alfonso
Embid-Desmet (@alfondotnet)
* Bump activesupport from 8.0.2 to 8.0.4.1 (#3270) via dependabot[bot]
(@dependabot[bot])
* Merge release PR after deploy (#3269) via Antonio Pallares
(@ajpallares)
* Require PR approval before release tagging (#3268) via Antonio
Pallares (@ajpallares)
* Bump json from 2.18.1 to 2.19.2 (#3261) via dependabot[bot]
(@dependabot[bot])
* feat(ads): update admob sample app (#3264) via Peter Porfy
(@peterporfy)
* feat(ads): add vanilla-ad-tracker-sample (#3263) via Peter Porfy
(@peterporfy)
* [Purchase Tester]: Persist appUserId on login screen across app
launches (#3266) via Will Taylor (@fire-at-will)

<!-- CURSOR_SUMMARY -->
---

> [!NOTE]
> **Medium Risk**
> Mostly release automation and versioning changes, but it modifies
CI/orb references and deploy/merge automation, which could affect the
release pipeline if misconfigured.
> 
> **Overview**
> Cuts the `9.28.0` release by removing `-SNAPSHOT` across version
sources (`.version`, `gradle.properties`, `Config.frameworkVersion`) and
updating sample/test-app dependency pins to `9.28.0`.
> 
> Updates release documentation: publishes Dokka docs to the `9.28.0` S3
path, updates `docs/index.html` redirect to `9.28.0`, and rolls
`CHANGELOG.latest.md` into a new `9.28.0` section in `CHANGELOG.md`.
> 
> Tweaks release tooling/CI: pins `fastlane-plugin-revenuecat_internal`
to a specific git ref (and bumps a few Ruby deps), switches the CircleCI
`revenuecat/sdks-common-config` orb to a dev commit, and adds a
temporary `test_merge_queue` workflow to exercise
`revenuecat/merge-release-pr` with `use_merge_queue: true`.
> 
> <sup>Written by [Cursor
Bugbot](https://cursor.com/dashboard?tab=bugbot) for commit
5050888. 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