saveourtool/diktat

`FILE_UNORDERED_IMPORTS` false positive when the list of imports contains an alias import

Open

#1494 aperta il 9 ago 2022

Vedi su GitHub
 (0 commenti) (0 reazioni) (1 assegnatario)Kotlin (40 fork)github user discovery
buggood first issue

Metriche repository

Star
 (571 star)
Metriche merge PR
 (Metriche PR in attesa)

Descrizione

Traditionally, IDEA places all alias imports into a separate group which can't be removed:

import-layout

On the contrary, diKTat doesn't distinguish between regular and alias imports and requires that all imports are sorted alphabetically.

As a workaround, the user has to suppress FILE_UNORDERED_IMPORTS whenever alias imports are used:

@file:Suppress("FILE_UNORDERED_IMPORTS")

package com.example

import java.lang.String

import org.assertj.core.api.Assertions.assertThat
import org.junit.jupiter.api.TestMethodOrder
import org.junit.jupiter.params.ParameterizedTest
import org.junit.jupiter.params.provider.ValueSource

import java.lang.Object as JvmObject

Environment information

  • diktat version: 1.2.3

Guida contributor

Direzione di ricerca
Esamina la logica di ordinamento delle importazioni in diktat per la regola FILE UNORDERED IMPORTS. Identifica dove le importazioni alias (es. `import ... as ...`) non vengono ordinate o raggruppate correttamente. Considera l'aggiunta di un'opzione di configurazione per trattare separatamente le importazioni alias o aggiornare il comparatore di ordinamento per gestire gli alias.
Tech stack
kotlin
Dominio
tooling
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
Kotlin
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
60