feat: Run lint and tests against both stable and min Flutter version#1707
Merged
rickvdl merged 2 commits intoApr 2, 2026
Merged
Conversation
tonidero
approved these changes
Apr 2, 2026
tonidero
left a comment
Contributor
There was a problem hiding this comment.
Makes sense! Thanks for doing this!
Looks like we will need to update required checks to merge this too. Lmk if you need help!
tonidero
reviewed
Apr 2, 2026
| name: Run tests | ||
| command: flutter test | ||
| name: Analyze code for purchases_ui_flutter | ||
| command: cd purchases_ui_flutter && flutter analyze lib |
Contributor
There was a problem hiding this comment.
One question I have though... The UI library depends on the core SDK. Do we need to build that first for the UI library to work? Or does this work automatically?
Member
Author
There was a problem hiding this comment.
It should resolve the core SDK package through the path as defined here, and AFAIK it should build the core SDK when building the UI SDK.
Reading it again the existing build-flutter-project step is a bit confusing because it doesn't actually build anything, but just runs flutter-get-dependencies. So I've replaced that for clarity
Parameterize CI lint and test jobs to run against both flutter:stable and the minimum supported Flutter version (3.22.0). Replaces the old test_main_sdk_min_version_compatibility and test_ui_sdk_min_version_compatibility jobs which only verified dependency resolution. Sample apps remain on stable only.
…rectly build-flutter-project was just a wrapper around flutter-get-dependencies with no additional logic. Replace all usages with the underlying command for clarity.
093985f to
ff2b112
Compare
RCGitBot
added a commit
that referenced
this pull request
Apr 9, 2026
**This is an automatic release.** ## RevenueCat SDK ### 📦 Dependency Updates * [AUTOMATIC BUMP] Updates purchases-hybrid-common to 17.55.1 (#1710) via RevenueCat Git Bot (@RCGitBot) * [Android 9.29.0](https://github.com/RevenueCat/purchases-android/releases/tag/9.29.0) * [iOS 5.67.1](https://github.com/RevenueCat/purchases-ios/releases/tag/5.67.1) * [AUTOMATIC BUMP] Updates purchases-hybrid-common to 17.55.0 (#1709) via RevenueCat Git Bot (@RCGitBot) ### 🔄 Other Changes * Bump fastlane-plugin-revenuecat_internal from `5d6e93f` to `6289be1` (#1708) via dependabot[bot] (@dependabot[bot]) * feat: Run lint and tests against both stable and min Flutter version (#1707) via Rick (@rickvdl) * Bump fastlane-plugin-revenuecat_internal from `f11fe40` to `5d6e93f` (#1706) via dependabot[bot] (@dependabot[bot]) <!-- CURSOR_SUMMARY --> --- > [!NOTE] > **Low Risk** > Primarily a version/release metadata bump with dependency version updates and no functional logic changes beyond reported plugin version strings. > > **Overview** > **Publishes release `9.16.1` across the Flutter SDK and RevenueCatUI plugin.** All package metadata and platform plugin constants are updated from `9.16.0` to `9.16.1` (Dart `pubspec.yaml`, Android `build.gradle` + `PurchasesFlutterPlugin.java`, iOS/macOS podspecs + `PurchasesFlutterPlugin.m`, and web plugin constant). > > Updates release documentation: adds the `9.16.1` section to `CHANGELOG.md`, refreshes `CHANGELOG-LATEST.md`, and adds a new row to `VERSIONS.md` pointing to `purchases-hybrid-common` `17.55.1` with iOS `5.67.1` and Android `9.29.0`. Also bumps `purchases_ui_flutter` to `9.16.1` and updates its dependency on `purchases_flutter` to `^9.16.1`. > > <sup>Reviewed by [Cursor Bugbot](https://cursor.com/bugbot) for commit 51d0ceb. Bugbot is set up for automated code reviews on this repo. Configure [here](https://www.cursor.com/dashboard/bugbot).</sup> <!-- /CURSOR_SUMMARY --> --------- Co-authored-by: Antonio Pallares <ajpallares@users.noreply.github.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Follow-up to #1704 which lowers the minimum Flutter SDK to 3.22.0.
Summary
Parameterize CI lint and test jobs to run against both
flutter:stableand the minimum supported Flutter version (3.22.0). Replaces the oldtest_main_sdk_min_version_compatibilityandtest_ui_sdk_min_version_compatibilityjobs which only verified dependency resolution.lint-testlint-test-min-versionlint-test-uilint-test-ui-min-versionlint-sample-appsTest plan
Note
Low Risk
Low risk: CI-only changes that adjust which CircleCI jobs run and which Flutter Docker images they use; main risk is longer runtimes or unexpected CI failures due to running tests on Flutter 3.22.0.
Overview
Updates CircleCI to run lint + unit tests for both
purchases_flutterandpurchases_ui_flutteragainstflutter:stableand the minimum supported Flutter version (3.22.0) via parameterizedlint-testandlint-test-uijobs.Replaces the old min-version compatibility jobs (which only resolved dependencies) with a new
lint-sample-appsjob for dependency resolution of sample apps/api_tester, and simplifies jobs by removing the unusedbuild-flutter-projectcommand and usingflutter-get-dependenciesdirectly in integration test steps.Written by Cursor Bugbot for commit ff2b112. This will update automatically on new commits. Configure here.