EasyEngine/easyengine

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

Open

#1,222 opened on 2018年9月14日

GitHub で見る
 (2 comments) (0 reactions) (0 assignees)PHP (429 forks)batch import
good first issue

Repository metrics

Stars
 (1,984 stars)
PR merge metrics
 (平均マージ 20d 14h) (30d で 7 merged PRs)

説明

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.

コントリビューターガイド

調査方針
リポジトリ内のphpcs.xmlファイルを見つけ、関数宣言後に空行を強制するルールを追加してください。具体的なスニフ（例：Squiz.WhiteSpace.FunctionSpacing）についてはPHPCSのドキュメントを参照してください。
技術スタック
php
領域
backenddevops
Issue 種別
機能
難度新しい貢献者にとっての実装難度です。1 は非常に小さな変更、5 は専門的な作業です。
1
推定時間経験ある貢献者が調査、実装、テスト、pull request 準備にかけるおおよその時間範囲です。
1時間未満
活動状況Issue が今どれだけ取り組みやすいかを示します。新しい、活発、古い、ブロック中、またはメンテナー入力待ちなどです。
アクティブ
明確さ期待される変更、受け入れ条件、次の手順がどれだけ明確かを示します。
明確
前提条件
PHPPHPCS
初心者向け度初回貢献者にどれだけ取り組みやすいかを 1-100 で推定したスコアです。
90