Repository navigation
fix(ci): build the Play internal bundle from the committed Android project - #332
Conversation
|
No actionable comments were generated in the recent review.
|
| Layer / File(s) | Summary |
|---|---|
Remove Android project initialization .github/workflows/publish.yaml |
The job no longer runs the Android initialization command or sets NDK environment variables for it. |
Priority:
Estimated code review effort: 1 (Trivial) | ~3 minutes
Change: Bug fix
Merge Risk: 
Minimal · up to 38380
-
The internal publishing job can use the committed Android project without the removed initialization step. No actionable merge risk remains after normal checks.
Pre-merge checks: 
5
-
Full details: Docstring Coverage
-
Explanation
No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0 files.
-
Finishing Touches
-
🧪 Generate unit tests (beta)
-
- Commit to this branch
- Create a new PR
-
- Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts
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 @coderabbitai help to get the list of available commands.
|
Automated deployment preview for the PR in the Cloudflare Pages. |
The Google Play internal job fails on every push to main: Gradle stops with "property 'projectDir' doesn't have a configured value" in
:app:rustBuildArm64Release.The job runs
tauri android init --cibefore building. With the current Tauri CLI, init overwrites the committedbuildSrc/.../BuildTask.ktwith a newer template that expects aprojectDir, but it keeps the committedRustPlugin.kt, which never sets one. Thepublish-androidjob builds the same committed project without init and succeeds, so this job now does the same.Summary by CodeRabbit