EasyEngine/easyengine

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

Open

#1.222 aberto em 14 de set. de 2018

Ver no GitHub
 (2 comments) (0 reactions) (0 assignees)PHP (429 forks)batch import
good first issue

Métricas do repositório

Stars
 (1.984 stars)
Métricas de merge de PR
 (Mesclagem média 20d 14h) (7 fundiu PRs em 30d)

Description

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.

Guia do colaborador

Direção de pesquisa
Localize o arquivo phpcs.xml no repositório e adicione uma regra para forçar uma linha em branco após as declarações de função. Consulte a documentação do PHPCS para o sniff específico (ex.: Squiz.WhiteSpace.FunctionSpacing).
Pilha de tecnologia
php
Domain
backenddevops
Tipo Issue
Funcionalidade
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
1
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
Menos de 1 hora
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
PHPPHPCS
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
90