Skip to content

update: rewrite of ksp-quickstart.md#5459

Merged
Caro Silva Rode (budindepunk) merged 10 commits into
masterfrom
ksp-quickstart
Apr 8, 2026
Merged

update: rewrite of ksp-quickstart.md#5459
Caro Silva Rode (budindepunk) merged 10 commits into
masterfrom
ksp-quickstart

Conversation

@budindepunk

Copy link
Copy Markdown
Collaborator

[KEDOC-100]

New quickstart for KSP based on this example in the KSP repository.

[[KEDOC-100](https://youtrack.jetbrains.com/issue/KEDOC-100/Update-the-KSP-quickstart)]

New quickstart for KSP based on [this example](google/ksp#2804) in the KSP repository.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Good start!

Let's first implement the changes I mentioned in the comments, and then I'll do a second, more thorough round.

Please pay special attention to the comments and apply them across the whole project, not only where I mentioned them explicitly.

Comment thread docs/topics/ksp/ksp-quickstart.md Outdated
Comment thread docs/topics/ksp/ksp-quickstart.md Outdated
Comment thread docs/topics/ksp/ksp-quickstart.md Outdated
Comment thread docs/topics/ksp/ksp-quickstart.md Outdated
Comment thread docs/topics/ksp/ksp-quickstart.md Outdated
Comment thread docs/topics/ksp/ksp-quickstart.md Outdated
Comment thread docs/topics/ksp/ksp-quickstart.md Outdated
Comment thread docs/topics/ksp/ksp-quickstart.md Outdated
Comment thread docs/topics/ksp/ksp-quickstart.md Outdated
Comment thread docs/topics/ksp/ksp-quickstart.md Outdated

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Added a bit more comments to the PR.

Comment thread docs/topics/ksp/ksp-quickstart.md Outdated
Comment thread docs/topics/ksp/ksp-quickstart.md
Comment thread docs/topics/ksp/ksp-quickstart.md Outdated
Comment thread docs/topics/ksp/ksp-quickstart.md Outdated
Comment thread docs/topics/ksp/ksp-quickstart.md
Comment thread docs/topics/ksp/ksp-quickstart.md Outdated
Comment thread docs/topics/ksp/ksp-quickstart.md Outdated
Comment thread docs/topics/ksp/ksp-quickstart.md Outdated
Comment thread docs/topics/ksp/ksp-quickstart.md Outdated
Comment thread docs/topics/ksp/ksp-quickstart.md Outdated
@@ -1,430 +1,458 @@
[//]: # (title: KSP quickstart)
[//]: # (title: Getting started with KSP)
[//]: # (description: Add an annotation processor based on Kotlin Symbol Processing (KSP) to your project, or use the KSP API to create your own.)

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

IDK why, but on the staging I could see a part of the phrase:

KSP API to create your own.)

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Just the last two comments and we are ready to go!
Please, resolve them before merging!

Comment thread docs/topics/ksp/ksp-quickstart.md Outdated
@budindepunk Caro Silva Rode (budindepunk) merged commit 2d944b5 into master Apr 8, 2026
5 checks passed
@budindepunk Caro Silva Rode (budindepunk) deleted the ksp-quickstart branch April 8, 2026 08:00
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