-
Notifications
You must be signed in to change notification settings - Fork 176
Update badges in README and set version to 0.7.4-SNAPSHOT #326
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
- Set version to 0.7.4-SNAPSHOT excluding samples because of #302 - Simplify build.yml - Update badges in README
8278342 to
fe08d16
Compare
README.md
Outdated
| @@ -1,8 +1,10 @@ | |||
| # MCP Kotlin SDK | |||
|
|
|||
| [](https://repo1.maven.org/maven2/io/modelcontextprotocol/kotlin-sdk/) | |||
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.
The badge should link to the Maven Central page, that is, to https://central.sonatype.com/artifact/io.modelcontextprotocol/kotlin-sdk
| # MCP Kotlin SDK | ||
|
|
||
| [](https://repo1.maven.org/maven2/io/modelcontextprotocol/kotlin-sdk/) | ||
| [](http://kotlinlang.org) |
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.
The Kotlin badge isn’t needed, especially with a fixed version. Having too many badges makes them harder to read
| [](https://repo1.maven.org/maven2/io/modelcontextprotocol/kotlin-sdk/) | ||
| [](http://kotlinlang.org) | ||
| [](https://kotlinlang.org/docs/multiplatform.html) | ||
| [](http://java.com) |
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.
Why is this badge needed?
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.
👍
Update badges in README and set version to 0.7.4-SNAPSHOT
Not updating the version of sample projects because of #302
Motivation and Context
How Has This Been Tested?
Breaking Changes
Types of changes
Checklist
Additional context