EasyEngine/easyengine

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

Open

#1222 aperta il 14 set 2018

Vedi su GitHub
 (2 commenti) (0 reazioni) (0 assegnatari)PHP (429 fork)batch import
good first issue

Metriche repository

Star
 (1984 star)
Metriche merge PR
 (Merge medio 20g 14h) (7 PR mergiate in 30 g)

Descrizione

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.

Guida contributor

Direzione di ricerca
Trova il file phpcs.xml nel repository e aggiungi una regola per imporre una riga vuota dopo le dichiarazioni di funzione. Fai riferimento alla documentazione di PHPCS per lo sniff specifico (es. Squiz.WhiteSpace.FunctionSpacing).
Tech stack
php
Dominio
backenddevops
Tipo issue
Funzionalità
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
1
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Meno di 1 ora
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
PHPPHPCS
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
90