Fix some lint issues and reduce severity of UnusedResources
This commit is contained in:
committed by
Benoit Marty
parent
e21055f399
commit
0de0069b1d
@@ -6,7 +6,6 @@
|
||||
|
||||
package io.element.android.x.features.messages
|
||||
|
||||
import Avatar
|
||||
import androidx.compose.foundation.interaction.MutableInteractionSource
|
||||
import androidx.compose.foundation.layout.Arrangement
|
||||
import androidx.compose.foundation.layout.Box
|
||||
@@ -75,6 +74,7 @@ import com.airbnb.mvrx.compose.mavericksViewModel
|
||||
import io.element.android.x.core.compose.LogCompositions
|
||||
import io.element.android.x.core.compose.PairCombinedPreviewParameter
|
||||
import io.element.android.x.core.data.StableCharSequence
|
||||
import io.element.android.x.designsystem.components.avatar.Avatar
|
||||
import io.element.android.x.designsystem.components.avatar.AvatarData
|
||||
import io.element.android.x.features.messages.components.MessageEventBubble
|
||||
import io.element.android.x.features.messages.components.MessagesReactionsView
|
||||
|
||||
@@ -2,7 +2,6 @@
|
||||
|
||||
package io.element.android.x.features.roomlist.components
|
||||
|
||||
import Avatar
|
||||
import androidx.activity.compose.BackHandler
|
||||
import androidx.compose.foundation.layout.fillMaxWidth
|
||||
import androidx.compose.material.ContentAlpha
|
||||
@@ -38,6 +37,7 @@ import androidx.compose.ui.text.font.FontWeight
|
||||
import androidx.compose.ui.unit.sp
|
||||
import io.element.android.x.core.compose.LogCompositions
|
||||
import io.element.android.x.core.compose.textFieldState
|
||||
import io.element.android.x.designsystem.components.avatar.Avatar
|
||||
import io.element.android.x.designsystem.components.dialogs.ConfirmationDialog
|
||||
import io.element.android.x.features.roomlist.model.MatrixUser
|
||||
|
||||
|
||||
@@ -1,6 +1,5 @@
|
||||
package io.element.android.x.features.roomlist.components
|
||||
|
||||
import Avatar
|
||||
import androidx.compose.foundation.background
|
||||
import androidx.compose.foundation.clickable
|
||||
import androidx.compose.foundation.interaction.MutableInteractionSource
|
||||
@@ -37,6 +36,7 @@ import androidx.compose.ui.unit.LayoutDirection
|
||||
import androidx.compose.ui.unit.dp
|
||||
import androidx.compose.ui.unit.sp
|
||||
import com.google.accompanist.placeholder.material.placeholder
|
||||
import io.element.android.x.designsystem.components.avatar.Avatar
|
||||
import io.element.android.x.features.roomlist.model.RoomListRoomSummary
|
||||
|
||||
private val minHeight = 72.dp
|
||||
|
||||
@@ -1,3 +1,5 @@
|
||||
package io.element.android.x.designsystem.components.avatar
|
||||
|
||||
import android.util.Log
|
||||
import androidx.compose.foundation.background
|
||||
import androidx.compose.foundation.layout.Box
|
||||
@@ -16,7 +18,6 @@ import androidx.compose.ui.unit.sp
|
||||
import coil.compose.AsyncImage
|
||||
import io.element.android.x.designsystem.AvatarGradientEnd
|
||||
import io.element.android.x.designsystem.AvatarGradientStart
|
||||
import io.element.android.x.designsystem.components.avatar.AvatarData
|
||||
|
||||
@Composable
|
||||
fun Avatar(avatarData: AvatarData, modifier: Modifier = Modifier) {
|
||||
@@ -76,5 +77,3 @@ private fun InitialsAvatar(
|
||||
)
|
||||
}
|
||||
}
|
||||
|
||||
|
||||
|
||||
@@ -1,9 +1,9 @@
|
||||
package io.element.android.x.matrix.sync
|
||||
|
||||
import io.element.android.x.matrix.util.mxCallbackFlow
|
||||
import kotlinx.coroutines.CoroutineScope
|
||||
import kotlinx.coroutines.flow.Flow
|
||||
import kotlinx.coroutines.launch
|
||||
import mxCallbackFlow
|
||||
import org.matrix.rustcomponents.sdk.SlidingSyncState
|
||||
import org.matrix.rustcomponents.sdk.SlidingSyncView
|
||||
import org.matrix.rustcomponents.sdk.SlidingSyncViewRoomListObserver
|
||||
|
||||
@@ -1,3 +1,5 @@
|
||||
package io.element.android.x.matrix.util
|
||||
|
||||
import kotlinx.coroutines.channels.ProducerScope
|
||||
import kotlinx.coroutines.channels.awaitClose
|
||||
import kotlinx.coroutines.flow.callbackFlow
|
||||
|
||||
@@ -27,7 +27,7 @@
|
||||
<ignore path="**/src/debug/**" />
|
||||
</issue>
|
||||
|
||||
<issue id="UnusedResources" severity="error">
|
||||
<issue id="UnusedResources" severity="warning">
|
||||
<!-- Ignore unused resource in debug build type -->
|
||||
<ignore path="**/src/debug/**" />
|
||||
|
||||
@@ -112,6 +112,10 @@
|
||||
<!-- Notification -->
|
||||
<issue id="LaunchActivityFromNotification" severity="error" />
|
||||
|
||||
<!-- Compose -->
|
||||
<issue id="ComposableLambdaParameterNaming" severity="error" />
|
||||
<issue id="ComposableLambdaParameterPosition" severity="error" />
|
||||
|
||||
<!-- DI -->
|
||||
<!-- issue id="JvmStaticProvidesInObjectDetector" severity="error" /-->
|
||||
</lint>
|
||||
|
||||
Reference in New Issue
Block a user