Skip to content

fix(ci): publish cloudy-native alongside cloudy on release - #158

Merged
l2hyunwoo merged 1 commit into
mainfrom
fix/publish-cloudy-native
Jul 11, 2026
Merged

fix(ci): publish cloudy-native alongside cloudy on release#158
l2hyunwoo merged 1 commit into
mainfrom
fix/publish-cloudy-native

Conversation

@l2hyunwoo

@l2hyunwoo l2hyunwoo commented Jul 11, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • Fixes Publishing issue with 0.7.0 #157cloudy-native:0.7.0 is missing on Maven Central, breaking consumer builds with Could not find com.github.skydoves:cloudy-native:0.7.0
  • cloudy-native was split into its own module as part of the AGP 9 upgrade, and cloudy's androidMain now depends on it via implementation(project(":cloudy-native")), which surfaces as a com.github.skydoves:cloudy-native runtime dependency in the published cloudy-android POM
  • publish.yml only ever ran :cloudy:publishAllPublicationsToMavenCentral, so cloudy-native (which already has its own mavenPublishing config) was never actually published
  • This updates the release workflow to build and publish :cloudy-native alongside :cloudy

Test plan

  • actionlint passes on the updated workflow (verified locally)
  • Next release run publishes both cloudy and cloudy-native artifacts to Maven Central
  • cloudy-native:0.7.0 needs to be backfilled separately (e.g. ./gradlew :cloudy-native:publishAllPublicationsToMavenCentral --no-configuration-cache from the 0.7.0 tag) since this fix only covers future releases

Summary by CodeRabbit

  • New Features
    • Cloudy Native artifacts are now included in release builds and published alongside Cloudy artifacts.
  • Chores
    • Updated the release workflow to assemble and publish both available modules.

@l2hyunwoo
l2hyunwoo requested a review from skydoves as a code owner July 11, 2026 22:13
@coderabbitai

coderabbitai Bot commented Jul 11, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: e9f0e27a-5433-44d9-9f57-0bc26a8be528

📥 Commits

Reviewing files that changed from the base of the PR and between 53baa89 and bc14c8b.

📒 Files selected for processing (1)
  • .github/workflows/publish.yml

📝 Walkthrough

Walkthrough

The GitHub Actions publish workflow now assembles and publishes both the :cloudy and :cloudy-native artifacts to Maven Central.

Changes

Publishing workflow

Layer / File(s) Summary
Build and publish release artifacts
.github/workflows/publish.yml
The release workflow assembles both modules and publishes both Maven Central publications with --no-configuration-cache.

Estimated code review effort: 1 (Trivial) | ~5 minutes

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Linked Issues check ⚠️ Warning The workflow publishes cloudy-native going forward, but it does not backfill the missing 0.7.0 artifact required by #157. Publish or backfill com.github.skydoves:cloudy-native:0.7.0 separately, or narrow the issue scope to future releases only.
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly states the CI release change to publish cloudy-native alongside cloudy.
Description check ✅ Passed The description covers the goal, implementation, and test plan, with the missing template sections being non-critical here.
Out of Scope Changes check ✅ Passed The only code changes are the release workflow updates needed to publish cloudy-native, so nothing appears unrelated.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/publish-cloudy-native

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@github-actions

Copy link
Copy Markdown

Snapshot diff report

✅ 8 screenshots verified, no changes.

Total Unchanged Changed Added
8 8 0 0

Full screenshot report (artifact)

Screenshots

Committed reference screenshots each spec verifies against (current expected state).

File name Image
passthrough_radius8.png
smoke_radius0.png
passthrough_radius16.png
smoke_radius24.png
state_disabled.png
state_enabled.png
passthrough_radius0.png
passthrough_radius24.png

@l2hyunwoo
l2hyunwoo merged commit f8ac062 into main Jul 11, 2026
5 checks passed
@l2hyunwoo
l2hyunwoo deleted the fix/publish-cloudy-native branch July 11, 2026 22:28
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.

Publishing issue with 0.7.0

1 participant