EasyEngine/easyengine

Add phpcs rule to ensure one line is left empty after function declaration

Open

#1.222 geöffnet am 14. Sept. 2018

Auf GitHub ansehen
 (2 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)PHP (429 Forks)batch import
good first issue

Repository-Metriken

Stars
 (1.984 Stars)
PR-Merge-Metriken
 (Durchschn. Merge 20T 14h) (7 gemergte PRs in 30 T)

Beschreibung

Currently PHPCS does not flag this as warning(which it should) -

function abc($a) {
    if($a){
        ...
    }
}

This is the coding the team has agreed to(and it should not display warning) -

function abc($a) {

    if($a){
        ...
    }
}

phpcs.xml needs to be modified in this and all the commands and site type repos.

Contributor Guide

Research-Richtung
Finden Sie die phpcs.xml Datei im Repository und fügen Sie eine Regel hinzu, die nach Funktionsdeklarationen eine Leerzeile erzwingt. Lesen Sie die PHPCS Dokumentation für den spezifischen Sniff (z.B. Squiz.WhiteSpace.FunctionSpacing).
Tech Stack
php
Domain
backenddevops
Issue Type
Funktion
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
1
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
Unter 1 Stunde
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
PHPPHPCS
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
90