-
Notifications
You must be signed in to change notification settings - Fork 177
Kotlin-MCP can't be used without ktor-bom starting from 0.7.5 (#390) #392
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
There was a problem hiding this 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 fixes a dependency management issue where kotlin-mcp couldn't be used without the ktor-bom starting from version 0.7.5. The fix explicitly adds version references to all Ktor dependencies so they can be resolved independently.
Key Changes:
- Added explicit version references (
version.ref = "ktor") to all Ktor dependency declarations in the Gradle version catalog
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
Pom generated on current main HEAD: |
devcrocod
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Remove ktor-bom from the project dependencies
Motivation and Context
kotlin-mcp should be usable without applying ktor-bom #390
How Has This Been Tested?
By running generatePomFileForJvmPublication and validating pom:
Breaking Changes
No
Types of changes
Checklist
Additional context