Linebreaks in generics
I maintainer di solito rispondono entro 2 giorni
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 3/5
- Tempo stimato
- 1-2 giorni
- Idoneità per principianti
- 45/100
Direzione di ricerca
Inizia riproducendo l’input Java fornito con il formatter e confronta il suo output con il comportamento previsto. Segui il percorso di formattazione dei parametri di tipo generici di classi e metodi, quindi aggiungi un test di regressione per gli esempi; il lavoro è completato quando le liste generiche rimangono su una sola riga, mentre l’andata a capo esistente degli argomenti resta invariata.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
In testing an upgrade to 1.6.1, we're sometimes seeing line breaks introduced into generics. Seems like this is probably a result of #512. I can see this being useful in pathological cases with extremely long generic type lists, but the issue is that we run prettier with a pretty aggressive line length limit, so we end up seeing split generics even in relatively straight-forward code samples. And it's pretty jarring when you first see it, because as far as I know, splitting generics like this isn't an idiomatic way to format Java (at least I haven't seen it in the wild before).
I tested the prettier-java upgrade on ~1,500 of our internal GitHub repos, and it definitely seemed like, in most cases, this generic splitting was hurting the code readability more than it helped. But curious to get your thoughts on how attached you are to this feature.
Prettier-Java 1.6.1
# Options (if any):
Input:
public abstract class GenericWblGeneratorWithLongName<OBJECT_TYPE, RECORD_TYPE> {
public static <RECORD, OFFSET> PagedResult<RECORD, OFFSET> appendCustomOffsets(
Arg arg1,
Arg arg2
) {
// implementation
}
}
Output:
public abstract class GenericWblGeneratorWithLongName<
OBJECT_TYPE, RECORD_TYPE
> {
public static <
RECORD, OFFSET
> PagedResult<RECORD, OFFSET> appendCustomOffsets(Arg arg1, Arg arg2) {
// implementation
}
}
Expected behavior:
public abstract class GenericWblGeneratorWithLongName<OBJECT_TYPE, RECORD_TYPE> {
public static <RECORD, OFFSET> PagedResult<RECORD, OFFSET> appendCustomOffsets(
Arg arg1,
Arg arg2
) {
// implementation
}
}
- Lingua principale
- Java
- Stelle
- 1.2k
- Fork
- 122
- Merge medio
- 2g 13h
- PR unite (30g)
- 32
Preparare l'ambiente
- Nessun Dockerfile né file Docker Compose
- Ha un modello di pull request
- Leggi la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di jhipster/prettier-java
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
jhipster/prettier-java#1113 ·
I maintainer di solito rispondono entro 2 giorni
-
Difficoltà 3/5 1-2 giorni Idoneità per principianti 68/100
jhipster/prettier-java#1115 ·
I maintainer di solito rispondono entro 2 giorni
-
Difficoltà 3/5 1-2 giorni Idoneità per principianti 56/100
jhipster/prettier-java#1114 ·
I maintainer di solito rispondono entro 2 giorni
-
Difficoltà 3/5 1-2 giorni Idoneità per principianti 66/100
jhipster/prettier-java#1112 ·
I maintainer di solito rispondono entro 2 giorni
-
Comment "// @formatter:off" was not printedForse già presa @jtkiesel l’ha presa 1 giorno fa. Aperta
Difficoltà 4/5 3-5 giorni Idoneità per principianti 35/100
jhipster/prettier-java#1098 ·
I maintainer di solito rispondono entro 2 giorni
Tutte le issue di jhipster/prettier-java
Issue simili
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
QuiltMC/update-quilt-meta#28 ·
-
awaiting triage Documentation P3
Difficoltà 1/5 1-3 ore Idoneità per principianti 86/100
I maintainer di solito rispondono entro 1 giorno
-
area/build area/maintainability component/camunda-process-test kind/tech-debt
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
I maintainer di solito rispondono entro 1 giorno
-
Kafka: Filter empty strings and whitespace in IcebergSinkConfig stringToListForse già presa @mohamed-elbrik l’ha presa oggi. Apertabug
Difficoltà 2/5 1-3 ore Idoneità per principianti 68/100
I maintainer di solito rispondono entro 1 giorno
-
area/kotlin
Difficoltà 2/5 1-3 ore Idoneità per principianti 84/100
I maintainer di solito rispondono entro 1 giorno