Splitting of long lines with closures and imported use variables
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 3/5
- Tiempo estimado
- 1-2 días
- Aptitud para principiantes
- 45/100
Línea de trabajo
El issue no menciona archivos fuente ni tests. Empieza reproduciendo la entrada PHP proporcionada en el playground del plugin enlazado y, después, localiza la lógica del formateador para los parámetros de las closures y las variables use; la tarea estará terminada cuando la salida formateada mantenga las variables use junto con los argumentos de la función en la disposición preferida.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
The way closure imports ("use" variables) are split when a line is too long is not ideal. I'm wondering if there is a way the priority can be changed so that it prefers to keep use variables together with the function arguments?
@prettier/plugin-php v0.16.1
Playground link
Input:
<?php
function test()
{
expectNotification(VerifyEmail::class)->toBeSentTo(
$user,
function ($notification, $channels, $notifiable) use ($user, &$actionUrl) {
$actionUrl = $notification->toMail($user)->actionUrl;
return $channels === ['mail'] && $notifiable->is($user);
},
);
}
Output:
<?php
function test()
{
expectNotification(VerifyEmail::class)->toBeSentTo($user, function ($notification, $channels, $notifiable) use (
$user,
&$actionUrl
) {
$actionUrl = $notification->toMail($user)->actionUrl;
return $channels === ['mail'] && $notifiable->is($user);
});
}
Of course Javascript doesn't have an equivalent of "use" imports, but I tried to approximate it below, and Prettier does what I would expect/prefer.
Prettier 2.2.1
Playground link
--parser babel
--print-width 120
--single-quote
--tab-width 4
--trailing-comma all
Input:
function test()
{
expectNotification(VerifyEmail.name).toBeSentTo(
$user,
function ($notification, $channels, $notifiable, use, $user, A$actionUrl) {
$actionUrl = $notification.toMail($user).actionUrl;
return $channels === ['mail'] && $notifiable.is($user);
},
);
}
Output:
function test() {
expectNotification(VerifyEmail.name).toBeSentTo(
$user,
function ($notification, $channels, $notifiable, use, $user, A$actionUrl) {
$actionUrl = $notification.toMail($user).actionUrl;
return $channels === ['mail'] && $notifiable.is($user);
},
);
}
- Lenguaje dominante
- PHP
- Estrellas
- 1.9k
- Forks
- 139
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de prettier/plugin-php
-
PHP 8.3 not allowed in options? Abierto
Dificultad 1/5 Menos de una hora Aptitud para principiantes 62/100
prettier/plugin-php#2416 · 2 comentarios · 6 reacciones ·
-
Dificultad 3/5 1-2 días Aptitud para principiantes 64/100
prettier/plugin-php#2498 ·
-
Dificultad 4/5 3-5 días Aptitud para principiantes 55/100
prettier/plugin-php#2495 ·
-
Dificultad 3/5 1-2 días Aptitud para principiantes 55/100
prettier/plugin-php#2494 · 1 comentario ·
-
strange format issue Abierto
Dificultad 3/5 1-2 días Aptitud para principiantes 48/100
prettier/plugin-php#2490 ·
Todos los issues de prettier/plugin-php
Issues similares
-
tooling
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
-
UX
Dificultad 2/5 1-3 horas Aptitud para principiantes 70/100
ProfessionalWiki/NeoWiki#1525 ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 90/100
OpenConext/OpenConext-engineblock#2122 ·
-
Bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 70/100
Automattic/safe-publish#594 ·