-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
KTOR-6501 Add feedback link to KDocs #4531
Conversation
ktor-client/ktor-client-android/jvm/src/io/ktor/client/engine/android/Android.kt
Outdated
Show resolved
Hide resolved
71c1a56
to
30d5fdc
Compare
30d5fdc
to
1011d91
Compare
1011d91
to
f6ae7af
Compare
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 only question bothering me is why the "Report a problem" button is generated not for all class members.
ktor-client/ktor-client-cio/common/src/io/ktor/client/engine/cio/CIOEngineConfig.kt
Show resolved
Hide resolved
It would be real cool if we could generate this through some KDoc extension instead of adding it everywhere in the code. |
Looks like you've incurred the wrath of CodeStyle for touching all the files 😄 |
@bjhham it is better to have it in the source code, cause the link will also be available through quick help of IDE |
4c710ae
to
a5bf930
Compare
a5bf930
to
5822465
Compare
Fix https://youtrack.jetbrains.com/issue/KTOR-6501