PowerShell / PowerShell/PSScriptAnalyzer

feature request for formatting rule to have an empty line after brace

オープン
#1,679 コメント 2 件 リアクション 0 件 担当者 0 名 GitHub で見る

まだ誰も着手していません。

Area - Formatter Issue - Enhancement Up-for-Grabs
主要言語
C#
スター
2.2k
フォーク
414
平均マージ
13時間 1分
マージ済み PR(30日)
2

説明

Summary of the new feature
This is a proposal to add additional formatting rule to have an empty line after brace to keep code more readable with OTBS scheme. It's a separate rule comparing to inserting a new line.

Current behaviour:

...
foreach ($account in $Permissions.Keys) {
    foreach ($permission in $Permissions[$account].Keys) {
        foreach ($folder in $Permissions[$account][$permission]) {
            cNtfsPermissionEntry $folder {
                Ensure                   = 'Present'
                Path                     = $folder
                Principal                = $account
                AccessControlInformation = @(
                    cNtfsAccessControlInformation {
                        FileSystemRights  = $permission
                    }
                )
            }
        }
    }
}
...

Proposed behaviour:

...
foreach ($account in $Permissions.Keys) {

    foreach ($permission in $Permissions[$account].Keys) {

        foreach ($folder in $Permissions[$account][$permission]) {

            cNtfsPermissionEntry $folder {
                Ensure                   = 'Present'
                Path                     = $folder
                Principal                = $account
                AccessControlInformation = @(
                    cNtfsAccessControlInformation {
                        FileSystemRights  = $permission
                    }
                )
            }
        }
    }
}
...

コントリビューションガイド

コントリビューションガイドを開く

はじめの一歩

  1. issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
  2. 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
  3. リポジトリをフォークし、ブランチを切って変更します。
  4. issue 番号を参照したプルリクエストを送ります。

調査の方向性

まず PSScriptAnalyzer のフォーマット規則と波括弧の処理ロジックを特定し、次にフォーマット動作に関する既存のテストを確認します。独立したルールによって波括弧の後に提案された空行が生成され、テストで示された入れ子になった PowerShell の例がカバーされれば完了です。

索引モデルが issue の本文から書いたものです。

評価

技術スタック
powershell
領域
tooling
issue の種類
機能追加
難易度
5/5
見積もり時間
1週間以上
活発さ
停滞
明瞭さ
おおむね明確
初心者へのやさしさ
35/100

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。