feat(feature:loan): migrated to CMP #2786
Conversation
| @@ -7,6 +7,8 @@ | |||
| * | |||
| * See https://github.com/openMF/mobile-mobile/blob/master/LICENSE.md | |||
| */ | |||
| @file:Suppress("ktlint:standard:property-naming") | |||
There was a problem hiding this comment.
Hi @Nagarjuna0033 ,
I see that you used @file:Suppress("ktlint:standard:property-naming") at the top of the file. Just a quick note when this annotation is placed at the file level, it suppresses all property naming violations in the entire file.
As we discussed before, when I ran it with @Suppress("PropertyName")on a specific property, I didn’t receive any Spotless errors. This approach could be a better choice since it only suppresses the rule for that particular property, and in the future, if someone adds a property that doesn’t follow naming conventions, it will still trigger a warning.
If @Suppress("PropertyName") doesn’t work for you, which worked for me, then you can try @Suppress("ktlint:standard:property-naming") directly on the specific property
| alias(libs.plugins.mifos.android.library.compose) | ||
| alias(libs.plugins.mifos.cmp.feature) | ||
| alias(libs.plugins.kotlin.serialization) | ||
| id(libs.plugins.kotlin.parcelize.get().pluginId) |
There was a problem hiding this comment.
Adding Pacelize plugin this way is cleaner, shorter and more readable
alias(libs.plugins.kotlin.parcelize)
Fixes - Jira-#180
Didn't create a Jira ticket, click here to create new.
Please Add Screenshots If there are any UI changes.
Please make sure these boxes are checked before submitting your pull request - thanks!
Run the static analysis check
./gradlew checkorci-prepush.shto make sure you didn't break anythingIf you have multiple commits please combine them into one commit by squashing them.