Skip to content

chore(build): upload debug APK artifacts in CI#613

Merged
sarim2000 merged 1 commit into
sarim2000:mainfrom
Reputable2772:main
Jul 19, 2026
Merged

chore(build): upload debug APK artifacts in CI#613
sarim2000 merged 1 commit into
sarim2000:mainfrom
Reputable2772:main

Conversation

@Reputable2772

Copy link
Copy Markdown
Contributor

Summary

Adds CI artifact uploads for debug APK builds so generated Android builds can be downloaded and tested directly from GitHub Actions runs.

Type of change

  • feat: New feature
  • fix: Bug fix
  • docs: Documentation update
  • refactor: Code improvement (no behavior change)
  • perf: Performance improvement
  • style: Formatting (no behavior change)
  • test: Add/update tests
  • chore/ci: Build or CI changes

Screenshots/Recordings (optional)

N/A

How to test

  1. Push a branch or open a pull request to trigger the GitHub Actions workflow.
  2. Verify the App Compile job successfully builds the standard debug APK.
  3. Download the standard-debug-apk artifact from the workflow run and verify the APK installs successfully.

Breaking changes

  • No breaking changes
  • Yes (describe):

Related issues

N/A

Checklist

  • Focused PR (single purpose)
  • Tests added/updated (if applicable)
  • Docs updated (if applicable)
  • ./gradlew test passes locally
  • ./gradlew lint passes locally
  • Follows existing code style and patterns

@greptile-apps

greptile-apps Bot commented Jul 17, 2026

Copy link
Copy Markdown
Contributor

Greptile Summary

This PR extends the app-compile CI job to run a full assembleDebug build and upload the resulting APKs (standard and fdroid flavor) as GitHub Actions artifacts with a 7-day retention window.

  • Adds ./gradlew :app:assembleDebug after the existing Kotlin-only compile check, then uploads both flavor APKs via actions/upload-artifact@v4.
  • Neither upload step sets if-no-files-found, so a path mismatch would produce an empty artifact and a green workflow rather than a visible failure.

Confidence Score: 5/5

Safe to merge — the change only adds artifact upload steps to an existing CI job with no impact on app code or test logic.

The change is additive and isolated to CI configuration. The new build and upload steps do not affect any application code, and the existing compile and test jobs are untouched. The only notable gap is that missing APK files would produce a silent warning rather than a job failure, which is a CI hygiene concern but not a blocking defect.

No files require special attention.

Important Files Changed

Filename Overview
.github/workflows/test.yml Adds assembleDebug build step and two APK artifact uploads (standard + fdroid flavors) to the app-compile job; artifact upload steps use the default if-no-files-found: warn which silently succeeds on missing files, and the existing 10-minute timeout may be too tight for a full APK build.

Sequence Diagram

%%{init: {'theme': 'neutral'}}%%
sequenceDiagram
    participant GH as GitHub Actions
    participant Job as app-compile job
    participant Gradle as Gradle
    participant Artifacts as Artifact Store

    GH->>Job: Trigger (push / PR)
    Job->>Gradle: compileStandardDebugKotlin --build-cache
    Gradle-->>Job: Kotlin compilation done
    Job->>Gradle: assembleDebug --build-cache
    Gradle-->>Job: "standard/debug/*.apk + fdroid/debug/*.apk"
    Job->>Artifacts: Upload standard-debug-apk (7 days)
    Job->>Artifacts: Upload fdroid-debug-apk (7 days)
    Job->>Artifacts: Upload compile-report (always, 7 days)
Loading
%%{init: {'theme': 'base', 'themeVariables': {"darkMode": true, "background": "#0d1117", "primaryColor": "#21262d", "primaryTextColor": "#e6edf3", "primaryBorderColor": "#8b949e", "lineColor": "#8b949e", "textColor": "#e6edf3", "edgeLabelBackground": "#161b22", "actorBkg": "#21262d", "actorBorder": "#8b949e", "actorTextColor": "#e6edf3", "actorLineColor": "#8b949e", "signalColor": "#8b949e", "signalTextColor": "#e6edf3", "noteBkgColor": "#373320", "noteBorderColor": "#d4a72c", "noteTextColor": "#f0e6c0", "labelBoxBkgColor": "#21262d", "labelBoxBorderColor": "#8b949e", "labelTextColor": "#e6edf3", "loopTextColor": "#e6edf3", "activationBkgColor": "#30363d", "activationBorderColor": "#8b949e"}}}%%
sequenceDiagram
    participant GH as GitHub Actions
    participant Job as app-compile job
    participant Gradle as Gradle
    participant Artifacts as Artifact Store

    GH->>Job: Trigger (push / PR)
    Job->>Gradle: compileStandardDebugKotlin --build-cache
    Gradle-->>Job: Kotlin compilation done
    Job->>Gradle: assembleDebug --build-cache
    Gradle-->>Job: "standard/debug/*.apk + fdroid/debug/*.apk"
    Job->>Artifacts: Upload standard-debug-apk (7 days)
    Job->>Artifacts: Upload fdroid-debug-apk (7 days)
    Job->>Artifacts: Upload compile-report (always, 7 days)
Loading

Reviews (2): Last reviewed commit: "chore(build): upload debug APK artifacts..." | Re-trigger Greptile

Comment thread .github/workflows/test.yml Outdated
Signed-off-by: Reputable2722 <153411261+Reputable2772@users.noreply.github.com>
@sarim2000
sarim2000 merged commit 74ece90 into sarim2000:main Jul 19, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants