Skip to content

Fix CI/CD pipeline and add CLAUDE.md #75

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.

Sign up for GitHub

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

Jump to bottom
Merged
thebehera merged 8 commits into from
Dec 19, 2025
Merged

Fix CI/CD pipeline and add CLAUDE.md #75

thebehera merged 8 commits into from
Dec 19, 2025

Conversation

@thebehera
Copy link
Contributor

@thebehera thebehera commented Dec 19, 2025 •
edited
Loading

Summary

  • Add CLAUDE.md for Claude Code guidance with build commands and architecture overview
  • Fix GPG signing issues in CI (SIGNING_PASSWORD secret was updated)
  • Run tests before publishing to Maven Central in merged.yaml
  • Handle existing tags gracefully in released.yaml
  • Speed up CI pipeline with job splitting, caching, and faster runners
  • Fix Android emulator hang after tests complete

Changes

CLAUDE.md

Provides guidance for Claude Code instances working in this repository:

  • Build commands (build, test, lint)
  • Architecture overview (Kotlin Multiplatform structure)
  • Platform-specific notes

CI/CD Fixes

  • merged.yaml: Added check task before publishToMavenCentral to ensure tests pass before publishing
  • released.yaml: Check if release exists before creating to avoid "tag already exists" errors
  • review.yaml: Removed redundant tasks, split into parallel jobs

CI/CD Speedups

  • review.yaml: Split into two parallel jobs:
    • Ubuntu: JVM, JS, WASM tests
    • macOS: Apple target tests (iOS/macOS)
  • Konan cache: Cache Kotlin/Native toolchains to avoid re-downloading
  • gradle.properties: Enable parallel builds and Gradle caching

Android Emulator Fix

Test plan

  • Verified GPG signing works with test workflow
  • Review workflow runs with split jobs (Apple + JVM/JS/WASM)
  • Android emulator tests pass for API 21 and API 29
  • Merged workflow publishes successfully after merge

@thebehera
Add CLAUDE.md for Claude Code guidance
6e5b2db
@thebehera
Fix CI/CD pipeline issues
9a9a35a 
- merged.yaml: Run tests (check) before publishing to Maven Central
- released.yaml: Skip release creation if tag already exists
- review.yaml: Remove redundant Gradle tasks (assemble, check already in build)
@thebehera
Speed up CI pipeline
ec423b4 
- Split review.yaml: Ubuntu for JVM/JS/WASM, macOS for Apple targets only
- Add Konan cache to avoid re-downloading Kotlin/Native toolchains
- Use macos-14 (M1) runners for faster Apple builds
- Enable Gradle parallel builds and caching
@thebehera
Simplify CI workflows
dba5ed9 
- Remove redundant gradle/actions/setup-gradle (setup-java cache is sufficient)
- Remove setup-chrome from merged.yaml (not needed for publishing)
- Remove publishToMavenLocal from review (not needed for PR testing)
- Add --no-build-cache for clean CI builds
@thebehera
Fix Android integration test hanging on post-action
499659a 
Remove gradle/actions/setup-gradle which hangs during build scan upload
@thebehera
Fix Android test hang: remove gradle cache, add timeout
50f3dfc
@thebehera
Use macos-latest instead of macos-14
f4f5d32
@thebehera
Fix Android emulator hang after tests complete
0739ed3 
Add cleanup step to kill crashpad_handler processes that prevent
the emulator from shutting down cleanly. This is a known issue
with reactivecircus/android-emulator-runner (see issue #385).
@thebehera thebehera merged commit ccd89e6 into main Dec 19, 2025
4 checks passed
@thebehera thebehera deleted the fix-ci-signing-v2 branch December 19, 2025 22:27
@thebehera thebehera restored the fix-ci-signing-v2 branch December 19, 2025 22:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@thebehera