Conversation
Scanned-by: gitleaks 8.29.0
📝 WalkthroughWalkthroughReplaced three dependency entries in Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes 🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches
🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
There was a problem hiding this comment.
Actionable comments posted: 1
🧹 Nitpick comments (1)
packages/integrations/rudder_integration_appsflyer_flutter/android/build.gradle (1)
30-30: Use an explicit version range forinstallreferrerto match other dependencies.Line 30 uses
2.+while all other dependencies (lines 22, 26, 29) use explicit lower-bound ranges. This introduces the same non-reproducibility concern. Update to[2.2, 3.0.0)following the established pattern.♻️ Proposed fix
- implementation 'com.android.installreferrer:installreferrer:2.+' + implementation 'com.android.installreferrer:installreferrer:[2.2, 3.0.0)'🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@packages/integrations/rudder_integration_appsflyer_flutter/android/build.gradle` at line 30, Replace the floating dependency version for com.android.installreferrer (the line using implementation 'com.android.installreferrer:installreferrer:2.+') with an explicit lower-bound range to match the project's pattern; update it to use the range [2.2, 3.0.0) so the dependency becomes an explicit, reproducible range.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In
`@packages/integrations/rudder_integration_appsflyer_flutter/android/build.gradle`:
- Line 29: Update the AppsFlyer Gradle dependency range to restrict automatic
upgrades to the current major version: replace the existing implementation line
referencing "com.appsflyer:af-android-sdk:[6.17.5, 17.0.0)" with a range that
caps at the next major, i.e., "[6.17.5, 7.0.0)"; ensure the change is made where
the dependency string appears so future patch/minor fixes are allowed but major
(breaking) upgrades are prevented.
---
Nitpick comments:
In
`@packages/integrations/rudder_integration_appsflyer_flutter/android/build.gradle`:
- Line 30: Replace the floating dependency version for
com.android.installreferrer (the line using implementation
'com.android.installreferrer:installreferrer:2.+') with an explicit lower-bound
range to match the project's pattern; update it to use the range [2.2, 3.0.0) so
the dependency becomes an explicit, reproducible range.
packages/integrations/rudder_integration_appsflyer_flutter/android/build.gradle
Outdated
Show resolved
Hide resolved
Scanned-by: gitleaks 8.29.0
1abhishekpandey
left a comment
There was a problem hiding this comment.
Let's specify the version in the PR title e.g.,: feat: bump the minimum appslfyer integration version to x.y.z.
packages/integrations/rudder_integration_appsflyer_flutter/android/build.gradle
Outdated
Show resolved
Hide resolved
…egration Scanned-by: gitleaks 8.29.0
|



This pull request updates the AppsFlyer integration dependencies in the Android build configuration to use a newer version of the RudderStack AppsFlyer integration, while removing direct dependencies on the AppsFlyer SDK and install referrer libraries.
Dependency updates:
com.rudderstack.android.integration:appsflyerdependency to use version range[2.4.0, 3.0.0)and removed direct dependencies oncom.appsflyer:af-android-sdkandcom.android.installreferrer.