Skip to content

Conversation

jclyne
Copy link
Contributor

@jclyne jclyne commented Oct 10, 2025

Changes to kotlin-sdk-core/build.gradle.kts to be compatible with gradle 9.x.x

Motivation and Context

If including this project into a composite build that uses gradle 9.x.x, recent changes to the build.gradle.kts are not compatible.

How Has This Been Tested?

The build still works with gradle 8.x

Breaking Changes

N/A

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

@jclyne jclyne marked this pull request as ready for review October 10, 2025 15:25
@Copilot Copilot AI review requested due to automatic review settings October 10, 2025 15:25
Copy link
Contributor

@Copilot 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 the Kotlin SDK build script to be compatible with Gradle 9.x.x by converting deprecated property assignment syntax to the newer .set() method syntax.

  • Updated library property assignment to use .set() method
  • Updated ignoreFileOverride property assignment to use .set() method

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Copy link
Contributor

@kpavlov kpavlov left a comment

Choose a reason for hiding this comment

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

Thank you, @jclyne

@kpavlov kpavlov merged commit 3d82883 into modelcontextprotocol:main Oct 11, 2025
3 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