feat:Added nlopez compose rules for ktlint and detekt #242
Conversation
Thank you for the pull request!The Scribe team will do our best to address your contribution as soon as we can. The 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 :) 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 General and Android rooms once you're in. Also consider joining our bi-weekly Saturday dev syncs. It'd be great to have you! Maintainer checklist |
|
Some linting issues were detected after running the ./gradlew lint command. However, these issues are related to XML files and can be addressed later, once the Jetpack Compose migration is complete. :) |
There was a problem hiding this comment.
All's working great here, @angrezichatterbox 😊 Thanks for devising this and getting to it so quickly! 🚀
Contributor checklist
./gradlew lintKotlin detekt testcommand as directed in the testing section of the contributing guideDescription
The pull request adds the
nlopezCompose rules to thektlintanddetekttasks to check for linting issues. It also corrects the existing linting issues that were detected. Additionally,Slack Compose ruleshave been added to the lint checks, ensuring that the IDE can detect any linting issues and provide recommendations for better practices.Related issue