fix: Fix dangling top-level kdoc comments (#390) #392
Conversation
Thank you for the pull request! ❤️The Scribe-Android team will do our best to address your contribution as soon as we can. If you're not already a member of our public Matrix community, please consider joining! We'd suggest using Element as your Matrix client, and definitely join the |
Maintainer ChecklistThe following is a checklist for maintainers to make sure this process goes as well as possible. Feel free to address the points below yourself in further commits if you realize that actions are needed :) |
|
@andrewtavis @angrezichatterbox could you please review this? 🙏 |
|
We'll get to the review as soon as we can, @majidhawa :) Thanks for the PR! |
There was a problem hiding this comment.
Thanks for the great work on the docstrings here, @majidhawa! Really has been such a major step for the app to have all of this work done, and will help so many others onboard in the future 😊
Contributor checklist
./gradlew lintKotlin detekt testcommand as directed in the testing section of the contributing guideDescription
This pull request adds full KDoc documentation to the
be.scri.views.KeyboardViewclass, including its public methods, properties, and nested interfaces.Additionally, it suppresses unused private property warnings using
@Suppress("UnusedPrivateProperty")to ensure a clean detekt pass while preserving future-use variables.Testing:
./gradlew detektsuccessfullyRelated issue
be.scri.views#376