-
Notifications
You must be signed in to change notification settings - Fork 2
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
Disable baseline profiles in release workflow #101
base: develop
Are you sure you want to change the base?
Conversation
WalkthroughThe pull request involves modifications to the GitHub Actions workflow file for Android Google Play Release, specifically focusing on the baseline profile generation step. The key change is the commenting out of the Gradle command responsible for generating release baseline profiles. This means that the process of creating baseline profiles, which typically helps optimize app performance, has been temporarily disabled. A TODO comment has been added, suggesting that there might be an ongoing issue or future task related to this workflow step. The rest of the workflow remains intact, including environment setup, code checkout, app bundle generation, artifact handling, Play Store upload, and Slack notification processes. The modification appears to be a deliberate pause in baseline profile generation, likely due to an underlying technical concern or investigation that requires further examination. The workflow's core functionality for releasing the Android application to Google Play remains unaffected by this specific change. The change is minimal and surgical, targeting only the baseline profile generation command without disrupting the overall release workflow structure. Tip CodeRabbit's docstrings feature is now available as part of our Early Access Program! Simply use the command Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media? 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
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.
Actionable comments posted: 0
🧹 Outside diff range and nitpick comments (1)
.github/workflows/android_google_play_release.yml (1)
Action
r0adkll/upload-google-play
should be updated from v1.1.1 to v1.1.3The current workflow uses version v1.1.1, while the latest available version is v1.1.3. Updating to the latest version is recommended to get bug fixes and improvements.
- Update
.github/workflows/android_google_play_release.yml
to user0adkll/[email protected]
🔗 Analysis chain
Line range hint
58-67
: Consider updating the Google Play upload actionThe workflow uses
r0adkll/[email protected]
which might have updates available. Consider checking for a newer version that might include bug fixes or improvements.Let's check the latest version:
🏁 Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash # Get the latest release version of the upload-google-play action gh api repos/r0adkll/upload-google-play/releases/latest --jq .tag_nameLength of output: 79
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (1)
.github/workflows/android_google_play_release.yml
(1 hunks)
🔇 Additional comments (2)
.github/workflows/android_google_play_release.yml (2)
36-37
: Verify the impact of disabling baseline profiles
The change aligns with the PR objective. However, disabling baseline profiles might affect app startup performance. Please ensure that:
- The performance impact has been measured and is acceptable
- This is a temporary solution with a plan to re-enable profiles once issue Baseline profiles don't work on CI #89 is resolved
Let's check if baseline profiles are used in other workflows:
✅ Verification successful
Temporarily disabling baseline profiles is acceptable
The code analysis reveals this is a valid temporary workaround for issue #89. The baseline profile setup is properly configured in the project:
- Baseline profile module is correctly set up with required dependencies
- Android app module has proper baseline profile integration
- The issue is specifically with CI environment execution, not the implementation
The performance impact of disabling profiles temporarily is an acceptable trade-off until the CI issue is resolved, as:
- Baseline profiles are only for optimization, not functionality
- The TODO comment properly tracks the issue
- The implementation remains intact for local development
🏁 Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash
# Check for baseline profile generation in other workflow files
rg -l "generateReleaseBaselineProfile" .github/workflows/
Length of output: 107
Script:
#!/bin/bash
# Check for baseline profile configuration in Gradle files
rg -l "baselineProfile" -g "*.gradle*"
# Check for any other baseline profile related files
fd "baseline.*profile"
# Check the content of issue #89 to understand the context
gh issue view 89
Length of output: 567
Script:
#!/bin/bash
# Check the baseline profile configuration in build files
cat androidApp/build.gradle.kts
cat baselineprofile/build.gradle.kts
# Check the baseline profile generator implementation
cat baselineprofile/src/main/kotlin/app/futured/kmptemplate/baselineprofile/BaselineProfileGenerator.kt
Length of output: 7296
Line range hint 11-14
: Address pending TODO configurations
Several configuration items need attention:
- Platform-specific Slack channel name is using a placeholder
- Product flavor configuration needs verification
- KMP_FLAVOR is set to 'prod' - verify if this is correct for release workflow
Let's check the available product flavors:
Generated by 🚫 Danger |
Due to #89
Summary by CodeRabbit