prettier / prettier/plugin-php

Format multiple-catch

Ouverte
#1,394 1 commentaire 2 réactions 0 personnes assignées Voir sur GitHub

Personne n'a encore pris cette issue.

enhancement
Langage dominant
PHP
Étoiles
1.9k
Forks
139
Métriques de merge des PR
Aucune PR mergée en 30 j

Description

We're currently not respecting line length limitations in multiple-catch blocks, which we could.


@prettier/plugin-php v0.14.0
Playground link

Input:

<?php

try {
    do_something();
} catch (\Some\Rather\Obscure\Exception | \Another\Obscure\Exception $exception) {
    handle_exception();
}

Current Output:

<?php

try {
    do_something();
} catch (\Some\Rather\Obscure\Exception | \Another\Obscure\Exception $exception) {
    handle_exception();
}

Improved Output:

<?php

try {
    do_something();
} catch (
    \Some\Rather\Obscure\Exception |
    \Another\Obscure\Exception $exception
) {
    handle_exception();
}


To be honest, the improved output is not perfect (the $exception variable is a little bit "obscured" next to the exception class), but it's better than the current behavior IMO.

Guide de contribution

Ouvrir le guide de contribution

Par où commencer

  1. Lisez l'issue en entier, puis le guide de contribution du projet.
  2. Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
  3. Forkez le dépôt et travaillez sur une branche.
  4. Ouvrez une pull request qui référence le numéro de l'issue.

Piste de recherche

Commencez par la reproduction fournie dans Playground et recherchez dans le PHP printer le formatage de multiple-catch ; l’issue ne nomme aucun fichier ni aucun test. Le travail est considéré comme terminé lorsque la liste des exceptions est répartie pour respecter la longueur de ligne configurée, tout en conservant la structure du catch-block présentée et avec les tests pertinents du formatter réussis.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Évaluation

Stack technique
php
Domaine
tooling
Type d'issue
Bug
Difficulté
3/5
Temps estimé
1-2 jours
Activité
À l'abandon
Clarté
Plutôt claire
Accessibilité débutants
48/100

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.