Skip to content

Conversation

@technoir42
Copy link

@technoir42 technoir42 commented Oct 6, 2025

Summary by CodeRabbit

  • New Features

    • Enables publishing releases to Maven Central automatically when a version tag is pushed.
  • Documentation

    • Updated README badge to show Maven Central version.
    • Revised dependency examples for Gradle Kotlin DSL and Kotlin Multiplatform.
  • Chores

    • Updated project coordinates (group and version) and repository links to the new organization.
    • Refreshed POM/metadata (URLs, SCM, license distribution) and removed outdated packaging config.
    • Added CI workflow to build and publish artifacts using tag-derived versioning.

@coderabbitai
Copy link

coderabbitai bot commented Oct 6, 2025

Caution

Review failed

The pull request is closed.

Walkthrough

Adds a GitHub Actions Deploy workflow for publishing on tag pushes, updates README badges and dependency snippet to Kotlin Multiplatform style, and revises Gradle project/POM metadata (group, version, URLs, developer info, and build settings) to the technoir-lab namespace.

Changes

Cohort / File(s) Summary of changes
CI/CD Deploy Workflow
.github/workflows/deploy.yaml
Added a Deploy workflow triggered on tag push and manual dispatch; sets concurrency; uses macos-latest; checks out code; sets up JDK 17 (Temurin); configures Gradle; caches Kotlin/Native toolchains at ~/.konan (keyed by OS + settings.gradle.kts); computes VERSION_NAME by stripping a leading v from the tag; runs gradle publish with secrets (CENTRAL_PORTAL_USER, CENTRAL_PORTAL_PASSWORD, SIGNING_KEY, SIGNING_PASSWORD).
Documentation
README.md
Replaced Maven Central badge with a Shields.io Maven Central version badge; replaced Gradle/Maven snippet with a Kotlin Multiplatform DSL example adding io.technoirlab:kotlin-math to commonMain; removed older Gradle repo/dependency blocks.
Build Metadata
gradle.properties
Updated project metadata: GROUP -> io.technoirlab, VERSION_NAME -> 1.7.0; tweaked POM_DESCRIPTION punctuation; updated POM_URL, POM_SCM_* to technoir-lab repo; removed POM_PACKAGING; added POM_LICENCE_DIST=repo and developer fields (POM_DEVELOPER_*); added Dokka plugin mode and several Gradle/Kotlin JVM/native flags and caching settings.

Sequence Diagram(s)

sequenceDiagram
  autonumber
  actor Dev as Developer
  participant GH as GitHub
  participant WF as Actions Runner
  participant GR as Gradle
  participant MC as Maven Central

  Dev->>GH: Push tag (e.g., v1.7.0) or manual dispatch
  GH-->>WF: Trigger Deploy workflow

  rect rgb(245, 250, 255)
    note over WF: Setup
    WF->>WF: Checkout repo
    WF->>WF: Set up JDK 17 (Temurin)
    WF->>WF: Configure Gradle
    WF->>WF: Restore/cache ~/.konan (key: OS + settings.gradle.kts)
    WF->>WF: Compute VERSION_NAME (strip leading 'v')
  end

  rect rgb(240, 255, 240)
    note over GR,MC: Publish
    WF->>GR: Run Gradle publish with secrets
    GR->>MC: Upload artifacts & POM
    MC-->>GR: Acknowledge
  end

  WF-->>GH: Report workflow result
Loading

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20–30 minutes

Possibly related PRs

  • Prepare for publishing #12 — Appears to implement the same publishing and metadata changes (deploy workflow, README updates, and matching gradle.properties edits).

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title Check ✅ Passed The title “Prepare for publishing” accurately reflects the primary purpose of the changeset, which is to add a deployment workflow and update project metadata for publishing; it is concise, clear, and directly tied to the main change without unnecessary detail.
Docstring Coverage ✅ Passed No functions found in the changes. Docstring coverage check skipped.

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 674a337 and ffba7c6.

📒 Files selected for processing (3)
  • .github/workflows/deploy.yaml (1 hunks)
  • README.md (2 hunks)
  • gradle.properties (1 hunks)

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

Copy link

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 2

📜 Review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between a458695 and 674a337.

📒 Files selected for processing (3)
  • .github/workflows/deploy.yaml (1 hunks)
  • README.md (2 hunks)
  • gradle.properties (1 hunks)

@technoir42 technoir42 merged commit e79e846 into main Oct 6, 2025
1 check 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