Skip to content

Fix maven central publishing#14

Merged
mateuszkwiecinski merged 1 commit intomasterfrom
updates
Feb 14, 2026
Merged

Fix maven central publishing#14
mateuszkwiecinski merged 1 commit intomasterfrom
updates

Conversation

@mateuszkwiecinski
Copy link
Copy Markdown
Member

No description provided.

Copilot AI review requested due to automatic review settings February 14, 2026 07:59
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR refactors the Maven Central publishing configuration to eliminate hardcoded description strings. The change moves the project description from the PublishingPlugin to the project-level build file, making it more maintainable and adding validation to ensure the description is always set.

Changes:

  • Added project description property to keeper-gradle-plugin build file
  • Refactored PublishingPlugin to use project.description instead of hardcoded strings
  • Added validation check to ensure project.description is set before publishing

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
keeper-gradle-plugin/build.gradle.kts Adds project description property for use in publishing metadata
gradle/plugins/src/main/kotlin/PublishingPlugin.kt Replaces hardcoded description with project.description and adds validation check

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@github-actions
Copy link
Copy Markdown

Diffuse output:

OLD: diffuse-base-file
NEW: keeper-gradle-plugin.jar

       │          compressed          │         uncompressed         
       ├───────────┬───────────┬──────┼───────────┬───────────┬──────
 JAR   │ old       │ new       │ diff │ old       │ new       │ diff 
───────┼───────────┼───────────┼──────┼───────────┼───────────┼──────
 class │ 109.3 KiB │ 109.3 KiB │  0 B │ 214.9 KiB │ 214.9 KiB │  0 B 
 other │   5.3 KiB │   5.3 KiB │  0 B │  11.3 KiB │  11.3 KiB │  0 B 
───────┼───────────┼───────────┼──────┼───────────┼───────────┼──────
 total │ 114.6 KiB │ 114.6 KiB │  0 B │ 226.2 KiB │ 226.2 KiB │  0 B 

 CLASSES │ old │ new │ diff      
─────────┼─────┼─────┼───────────
 classes │  72 │  72 │ 0 (+0 -0) 
 methods │ 553 │ 553 │ 0 (+0 -0) 
  fields │ 118 │ 118 │ 0 (+0 -0)

@mateuszkwiecinski mateuszkwiecinski merged commit 12694a0 into master Feb 14, 2026
9 of 11 checks passed
@mateuszkwiecinski mateuszkwiecinski deleted the updates branch February 14, 2026 08:06
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