Require PR approval before release tagging#41
Conversation
Add a `validate_pr_approved` check to `tag-current-branch` that verifies the release PR has been approved by an org member with write permissions before the tag is created. This prevents tagging a release before the PR is properly reviewed. Made-with: Cursor
Matches the Ruby version used by purchases-ios. Made-with: Cursor
Fixes RC009 lint: complex run step commands must use <<include()>> syntax. Made-with: Cursor
|
Your development orb has been published. It will expire in 30 days. |
Made-with: Cursor
|
Your development orb has been published. It will expire in 30 days. |
|
Your development orb has been published. It will expire in 30 days. |
1 similar comment
|
Your development orb has been published. It will expire in 30 days. |
|
Your orb has been published to the CircleCI Orb Registry. |
## 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. Additionally, the repo had a custom `tag-release-branch` job that was copy-pasted from `purchases-ios`, including a wrong `working_directory: ~/purchases-ios`. This PR cleans that up. ## Description - Bumps `revenuecat/sdks-common-config` orb to `@3.16.0`, which includes a `validate_pr_approved` step in the `tag-current-branch` job - Replaces the custom `tag-release-branch` job with the orb's `revenuecat/tag-current-branch` (passing `ruby_version: "3.3.0"`), since the custom job was functionally equivalent and had a wrong `working_directory` **Depends on:** RevenueCat/sdks-circleci-orb#41
## 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
## 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 <!-- CURSOR_SUMMARY --> --- > [!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. > > <sup>Written by [Cursor Bugbot](https://cursor.com/dashboard?tab=bugbot) for commit 604173c. This will update automatically on new commits. Configure [here](https://cursor.com/dashboard?tab=bugbot).</sup> <!-- /CURSOR_SUMMARY -->
## 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
## 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
## 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 <!-- CURSOR_SUMMARY --> --- > [!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. > > <sup>Written by [Cursor Bugbot](https://cursor.com/dashboard?tab=bugbot) for commit 1d68cd9. This will update automatically on new commits. Configure [here](https://cursor.com/dashboard?tab=bugbot).</sup> <!-- /CURSOR_SUMMARY -->
## 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
## 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
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
Adds a
validate_pr_approvedstep to thetag-current-branchjob. This verifies the release PR has been approved by an org member with write permissions before the git tag is created. If the PR is not approved, the job fails and the tag is not pushed.This change affects all SDK repos that use the
revenuecat/tag-current-branchorb job.Reference: RevenueCat/purchases-ios#6243
Made with Cursor