Windows should not be a plural noun

説明

Before submitting a bug report:

Make sure you are able to repro it on the latest released version
Perform a quick search for existing issues to check if this bug has already been reported

Steps to reproduce

See this tweet https://twitter.com/FrankLesniak/status/1453189852032344064?s=20

function Test-Windows {
dostuff
}

Expected behavior

PSUseSingularNouns warning is not thrown when noun finishes with Windows

Actual behavior

PSUseSingularNouns warning thrown

If an unexpected error was thrown then please report the full error details using e.g. $error[0] | Select-Object *

Environment data



Name                           Value
----                           -----
PSVersion                      7.2.0-preview.10
PSEdition                      Core
GitCommitId                    7.2.0-preview.10
OS                             Microsoft Windows 10.0.19043
Platform                       Win32NT
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0…}
PSRemotingProtocolVersion      2.3
SerializationVersion           1.1.0.1
WSManStackVersion              3.0



1.18.3 - Doesn't throw
1.20.0 - Does throw

はじめの一歩

issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。

着手することを issue にコメントします — 二人が同じ作業をするのを防げます。

リポジトリをフォークし、ブランチを切って変更します。

issue 番号を参照したプルリクエストを送ります。

調査の方向性

まず、報告されている PowerShell のバージョンで Test-Windows 関数を使って PSUseSingularNouns 警告を再現し、次に analyzer 内の PSUseSingularNouns ルールを追跡します。Windows という名詞に対して警告が出力されず、同時にルールが真に複数形の名詞を引き続き報告すれば完了です。

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

評価

技術スタック

csharp, powershell

領域

tooling

issue の種類

バグ

難易度

3/5

見積もり時間

1〜2日

活発さ

停滞

明瞭さ

おおむね明確

初心者へのやさしさ

45/100

説明