Skip to content

Require PR approval before release tagging#811

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

Require PR approval before release tagging#811
ajpallares merged 1 commit 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

@ajpallares ajpallares force-pushed the pallares/require-release-pr-approval branch from 6533752 to 7e8343d Compare March 20, 2026 17:42
@ajpallares ajpallares marked this pull request as ready for review March 23, 2026 11:24
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 7e8343d to 73d7353 Compare March 23, 2026 11:46
@ajpallares ajpallares requested a review from a team March 23, 2026 12:19
@ajpallares ajpallares enabled auto-merge (squash) March 23, 2026 12:19
@ajpallares ajpallares merged commit f15fa35 into main Mar 23, 2026
6 checks passed
@ajpallares ajpallares deleted the pallares/require-release-pr-approval branch March 23, 2026 12:23
tonidero pushed a commit that referenced this pull request Mar 24, 2026
**This is an automatic release.**

## RevenueCat SDK
### ✨ New Features
* Refactor wallet button renderer (#810) via Marek Dabek (@marek-dabek)
### 🐞 Bugfixes
* Add font-family to simulated store modal (#809) via Perttu
(@plahteenlahti)
* Fix price-per-period rounding to floor instead of round-half-up (#807)
via Dan Pannasch (@dpannasch)

### 🔄 Other Changes
* Require PR approval before release tagging (#811) via Antonio Pallares
(@ajpallares)
* PW-1121 | Add paywall events (impression, close, cancel) (#806) via
Drago Crnjac (@popcorn)
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