prettier / prettier/plugin-php

Single line conditions

Offen
#2,116 1 Kommentar 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

Vorherrschende Sprache
PHP
Sterne
1.9k
Forks
139
PR-Merge-Kennzahlen
Keine gemergten PRs in 30 T.

Beschreibung

When using simple conditions for early returns / guard statements the current formatting is quite verbose:

function test($arg1) {
  if (!$arg) return;

  for ($i = 0; $i < 10; $i++) {
    if ($i % 2 === 0) continue;
    // do something if the number is odd
  }
}

will be converted to:

function test($arg1) {
  if (!$arg) {
    return;
  }

  for ($i = 0; $i < 10; $i++) {
    if ($i % 2 === 0) {
      continue;
    }
    // do something if the number is odd
  }
}

It would be great if these single line conditions would be preserved. Alot of formatters for c-style languages like prettier for javascript or ClangFormat allow this, resulting in much more concise code.

Beitragsleitfaden

Beitragsleitfaden öffnen

Erste Schritte

  1. Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
  3. Forke das Repository und arbeite in einem Branch.
  4. Öffne einen Pull Request, der die Issue-Nummer nennt.

Rechercherichtung

Beginnen Sie mit den PHP-Beispielen im Issue und untersuchen Sie die Behandlung von einzeiligen frühen Rückgaben, Guard-Anweisungen und Schleifensteuerungen durch den Formatter. Ermitteln Sie, wie die aktuelle Blockerweiterung implementiert ist, und identifizieren Sie die relevanten bestehenden Tests, bevor Sie die erwartete beibehaltene Formatierung festlegen. Als abgeschlossen gilt dies, wenn diese einzeiligen Bedingungen prägnant bleiben und das übrige Formatierungsverhalten unverändert ist.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
php
Bereich
tooling
Issue-Typ
Feature
Schwierigkeit
4/5
Geschätzter Aufwand
3-5 Tage
Aktivitätsstatus
Veraltet
Klarheit
Größtenteils klar
Anfängerfreundlichkeit
42/100

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.