Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

MarkusBillharz
Copy link

This PR introduces minimal changes to allow contributors to run the sample projects against local builds of the SDK, without breaking the independence of the samples.

Resolves #196

Changes

  • Add mavenLocal() to the repositories block in the sample projects so they can resolve locally published artifacts.
  • Add a short note to the contributing guidelines explaining how to publish locally for testing using ./gradlew publishToMavenLocal
  • Increase the Kotlin version of the mcp-server-sample to get it to build (ABI error).
  • Ensure the sample projects are included in the PR checks to verify they build.

This avoids directly linking the samples to the main project (as discussed in #119 (review)), while making it easier to test local changes during development.

To test it, I fake version bumped it on my clone to check that it is actually working as intended:
https://github.com/MarkusBillharz/kotlin-sdk/actions/runs/16523835710/job/46731929589

Types of changes

  • Non-breaking changes
  • Documentation update

Feedback welcome!

@MarkusBillharz MarkusBillharz force-pushed the task/support-local-builds-in-samples branch from eeafc29 to 7ebe530 Compare August 1, 2025 09:40
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 @MarkusBillharz
That makes sense. Let's resolve conflicts and merge

@kpavlov kpavlov self-requested a review September 16, 2025 14:35
@kpavlov
Copy link
Contributor

kpavlov commented Sep 16, 2025

I have reproduced this PR here #275
Thank you for your help

@kpavlov kpavlov closed this Sep 16, 2025
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.

Samples don't seem to easily support testing against local SDK changes
2 participants