Fixed detekt warnings

wordpress-mobile · Sep 27, 2024 · 16e28ae · 16e28ae
1 parent 9e85e4a
commit 16e28ae
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 1 deletion.
diff --git a/...Press/src/main/java/org/wordpress/android/ui/mysite/items/listitem/SiteListItemBuilder.kt b/...Press/src/main/java/org/wordpress/android/ui/mysite/items/listitem/SiteListItemBuilder.kt
@@ -147,7 +147,7 @@ class SiteListItemBuilder @Inject constructor(
     }
 
     fun buildSelfHostedUserListItemIfAvailable(site: SiteModel, onClick: (ListItemAction) -> Unit): ListItem? {
-        // TODO: Should this excluded JetPack users?
+        // TODO Should this excluded JetPack users?
         return if (selfHostedUsersFeatureConfig.isEnabled() && site.selfHostedSiteId > 0) {
             ListItem(
                 R.drawable.ic_user_white_24dp,

diff --git a/WordPress/src/main/java/org/wordpress/android/ui/selfhostedusers/SampleUsers.kt b/WordPress/src/main/java/org/wordpress/android/ui/selfhostedusers/SampleUsers.kt
@@ -70,6 +70,7 @@ object SampleUsers {
         url = "example.com",
     )
 
+    @Suppress("MagicNumber")
     fun getSampleUsers(): ArrayList<UserWithEditContext> {
         if (sampleUserList.isEmpty()) {
             repeat(25) {

diff --git a/WordPress/src/main/java/org/wordpress/android/ui/selfhostedusers/UserListViewModel.kt b/WordPress/src/main/java/org/wordpress/android/ui/selfhostedusers/UserListViewModel.kt
@@ -27,6 +27,7 @@ class UserListViewModel @Inject constructor(
     val users = _users.asStateFlow()
 
     // TODO this uses dummy data for now - no network request is involved yet
+    @Suppress("MagicNumber")
     fun fetchUsers() {
         showProgressDialog(R.string.loading)
         _users.value = listOf()