Prepare next version: 5.71.0-SNAPSHOT#6719
Conversation
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, have a team admin enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit e0fb3dd. Configure here.
| <html> | ||
| <head> | ||
| <meta http-equiv="refresh" content="0; url=https://revenuecat.github.io/purchases-ios-docs/5.70.0/documentation/revenuecat/v4_api_migration_guide"/> | ||
| <meta http-equiv="refresh" content="0; url=https://revenuecat.github.io/purchases-ios-docs/5.71.0-SNAPSHOT/documentation/revenuecat/v4_api_migration_guide"/> |
There was a problem hiding this comment.
Docs index.html not updated in version bump
Low Severity
The version bump updated v4_api_migration_guide.html and v5_api_migration_guide.html to point to 5.71.0-SNAPSHOT, but the sibling scripts/docs/index.html still points to 5.70.0. All three files in scripts/docs/ serve the same purpose (redirecting to versioned documentation) and are expected to reference the same version. This inconsistency means index.html was likely missed during the version bump.
Additional Locations (1)
Reviewed by Cursor Bugbot for commit e0fb3dd. Configure here.
* Only run snapshot-bump workflow when `.version` changed The `snapshot-bump` workflow used to run on every non-scheduled, non-`bump` commit on `main`. Combined with `create_next_snapshot_version`'s only de-duplication being "is the bump branch currently on the remote?", any commit that landed on `main` between a release and the merge of its follow-up bump PR could open a duplicate bump PR (e.g. #6720 was opened 20s after #6719 merged because `bump/<v>` had just been deleted). Compute `version_file_changed` in the dynamic-config setup job by diffing the triggering commit against its parent for `.version`, forward it via `continuation/continue`'s `parameters` arg, and add it as a `when` condition on the `snapshot-bump` workflow. Now the workflow only triggers on commits that actually modified `.version` — i.e. the release commit itself (where the lane runs the bump) and the bump merge commit (where the lane no-ops because `.version` is now a SNAPSHOT). Unrelated commits on `main` no longer race against an in-flight bump PR. Made-with: Cursor * Move continuation-parameter computation to a Node script Mirrors the existing `.circleci/generate-requested-jobs-config.js` pattern so future derived pipeline parameters can be added without growing the inline shell in `.circleci/config.yml`. Made-with: Cursor * Drop now-redundant comment above continuation-parameters step The Node script's name and its own comments already convey what this step does. Made-with: Cursor * Drop redundant comments in default_config.yml The parameter name is self-explanatory and the rationale already lives in the generate-continuation-parameters.js script and commit history. Made-with: Cursor


Note
Low Risk
Low risk: this PR only updates version strings/metadata in podspecs, plists, internal
frameworkVersion, and docs redirect URLs with no functional code changes.Overview
Prepares the next SDK snapshot by bumping version metadata from
5.70.0to5.71.0-SNAPSHOT(andCFBundleShortVersionStringto5.71.0) across.version, both podspecs, andSystemInfo.frameworkVersion.Updates the docs redirect pages for the v4/v5 migration guides to point at the
5.71.0-SNAPSHOTdocumentation URLs.Reviewed by Cursor Bugbot for commit e0fb3dd. Bugbot is set up for automated code reviews on this repo. Configure here.