-
Notifications
You must be signed in to change notification settings - Fork 3
feat: Android 21 compatibility #188
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
echo "SDK Branch: android-sdk@${SDK_BRANCH}" | ||
echo "API Level: ${{ matrix.api-level }}" | ||
- name: Check out Common Java SDK |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
temporary until java.v4 is published (which we'll do when this and the java-server PRs are ready for merge)
- name: Publish Common SDK to Maven Local | ||
working-directory: sdk-common-jdk | ||
run: ./gradlew publishToMavenLocal --no-daemon | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
build the common SDK and publish to a local repository for the Android SDK
strategy: | ||
matrix: | ||
api-level: [26, 31, 34] | ||
api-level: [21, 26, 31, 34] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
big change!
defaultConfig { | ||
namespace "cloud.eppo.android" | ||
minSdk 26 | ||
minSdk 21 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
drop min to Android 21
Eppo Internal:
🎟️ Towards FF-4348