Skip to content

Conversation

@devcrocod
Copy link
Contributor

Motivation and Context

How Has This Been Tested?

Breaking Changes

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Documentation update

Checklist

  • I have read the MCP Documentation
  • My code follows the repository's style guidelines
  • New and existing tests pass locally
  • I have added appropriate error handling
  • I have added or updated documentation as needed

Additional context

@devcrocod devcrocod requested review from Copilot and tiginamaria July 18, 2025 18:16
Copy link
Contributor

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 updates CI workflow configuration for better debugging capabilities when publishing to Maven Central. The changes focus on enhancing visibility into the publication process and improving troubleshooting when issues occur.

  • Simplifies and updates Gradle configuration for better performance
  • Adds comprehensive debugging and error reporting for Maven Central publication failures
  • Introduces artifact upload for JReleaser logs to aid in troubleshooting

Reviewed Changes

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

File Description
gradle.properties Streamlines Gradle configuration by enabling parallel builds and consolidating JVM arguments
.github/workflows/gradle-publish.yml Enhances CI workflow with detailed logging, failure diagnostics, and artifact collection

@devcrocod devcrocod requested a review from sdubov July 19, 2025 08:35
Copy link
Contributor

@sdubov sdubov left a comment

Choose a reason for hiding this comment

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

I have one small question about printing the $AUTH variable. Other than that, LGTM.

@devcrocod devcrocod requested a review from sdubov July 21, 2025 09:56
sdubov
sdubov previously approved these changes Jul 21, 2025
@devcrocod devcrocod force-pushed the devcrocod/validate-publish-errors branch from 2f5724c to 50e5cb8 Compare July 21, 2025 10:47
@devcrocod devcrocod enabled auto-merge (squash) July 21, 2025 11:08
@devcrocod devcrocod merged commit da9def9 into main Jul 21, 2025
2 of 3 checks passed
@devcrocod devcrocod deleted the devcrocod/validate-publish-errors branch July 21, 2025 11:14
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.

4 participants