Document all functions in the helpers directory#352
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 :) |
|
Thanks for the PR @Femimat. We will get to review it soon :) |
angrezichatterbox
left a comment
There was a problem hiding this comment.
Everything looks good to me @Femimat
@andrewtavis I think we could add doc strings rule for the files in detekt and ktlint and then create issues and further start enabling them for different files.
andrewtavis
left a comment
There was a problem hiding this comment.
Looks great to me as well, @Femimat! Thanks for all the hard work to get the application documented for the community! 😊
|
I'll make an issue for detekt and ktlint now :) |
Contributor checklist
./gradlew lintKotlin detekt testcommand as directed in the testing section of the contributing guideDescription
The PR adds doc comments to all the functions and classes in the helpers directory.
Related issue