Skip to content

Release 1.8.x#1881

Merged
Prafulrakhade merged 3 commits into
mosip:release-1.8.xfrom
prathmeshj12:release-1.8.x
May 19, 2026
Merged

Release 1.8.x#1881
Prafulrakhade merged 3 commits into
mosip:release-1.8.xfrom
prathmeshj12:release-1.8.x

Conversation

@prathmeshj12
Copy link
Copy Markdown
Collaborator

@prathmeshj12 prathmeshj12 commented May 19, 2026

Summary by CodeRabbit

  • Chores
    • Updated publishing configuration for release distribution processes.

Review Change Stack

prathmeshj12 and others added 3 commits May 19, 2026 14:41
Signed-off-by: prathmeshj12 <166711249+prathmeshj12@users.noreply.github.com>
Signed-off-by: Prathmesh Jadhav <prathmeshjadhav@Prathmeshs-MacBook-Air.local>
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented May 19, 2026

Caution

Review failed

Pull request was closed or merged during review

Walkthrough

The api-test/pom.xml is updated to enable Maven Central publishing. A new property defines the central-publishing-maven-plugin version, and a distributionManagement block routes snapshot and release artifacts to Sonatype Central. The plugin is wired into the build pipeline with server authentication disabled for autoPublish.

Changes

Maven Central Publishing Configuration

Layer / File(s) Summary
Maven Central publishing property and plugin configuration
api-test/pom.xml
Maven property central.publishing.maven.plugin.version is added. DistributionManagement block defines snapshot and release repository endpoints for Sonatype Central. The central-publishing-maven-plugin is injected into the build plugins section with publishingServerId set to ossrh and autoPublish disabled.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

Poem

A rabbit hops to Maven's gate,
With plugins aligned and repos straight,
To Sonatype's keep, the code shall fly,
No auto-publish, just a watchful eye! 🐰📦

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 inconclusive)

Check name Status Explanation Resolution
Title check ❓ Inconclusive The title 'Release 1.8.x' is vague and generic, not describing the actual changes made to the codebase. Use a more specific title that describes the main change, such as 'Add Sonatype Central publishing configuration' or 'Configure Maven central publishing plugin for 1.8.x release'.
✅ Passed checks (4 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

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 and usage tips.

@Prafulrakhade Prafulrakhade merged commit 8f662ca into mosip:release-1.8.x May 19, 2026
16 of 17 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