◐ Shell
clean mode source ↗

Add rule for missing process block when command supports the pipeline by mattmcnabb · Pull Request #1373 · PowerShell/PSScriptAnalyzer

@mattmcnabb

@mattmcnabb mentioned this pull request

Nov 16, 2019
…nglyTypedCsFileForResx.ps1 Rules'

bergmeister

bergmeister

bergmeister

@bergmeister

@bergmeister

@mattmcnabb mattmcnabb changed the title WIP: Add rule for missing process block when command supports the pipeline Add rule for missing process block when command supports the pipeline

Nov 16, 2019

@mattmcnabb

bergmeister

bergmeister

bergmeister

bergmeister

@mattmcnabb

Christoph Bergmeister added 2 commits

November 22, 2019 07:42

bergmeister

bergmeister

bergmeister

bergmeister

Co-Authored-By: Christoph Bergmeister [MVP] <c.bergmeister@gmail.com>
Co-Authored-By: Christoph Bergmeister [MVP] <c.bergmeister@gmail.com>
Co-Authored-By: Christoph Bergmeister [MVP] <c.bergmeister@gmail.com>

rjmholt

Co-Authored-By: Robert Holt <rjmholt@gmail.com>
Co-Authored-By: Robert Holt <rjmholt@gmail.com>

bergmeister added a commit to thomasrayner/PSScriptAnalyzer that referenced this pull request

Dec 9, 2019

bergmeister pushed a commit that referenced this pull request

Dec 9, 2019
* avoidoverwritingbuiltincmdlets first draft

* rough draft avoidoverwritingcmdlets working

* Added tests, fixed typos, changed default PowerShellVersion behavior

* updates a/p rjmholt

* remove unneeded else

* avoidoverwritingbuiltincmdlets first draft

* rough draft avoidoverwritingcmdlets working

* Added tests, fixed typos, changed default PowerShellVersion behavior

* updates a/p rjmholt

* remove unneeded else

* updated readme - want tests to run in CI again

* prevent adding duplicate keys

* return an empty list instead of null

* update rule count

* fixing pwsh not present issue in test

* fixing a ps 4 test broke a linux test

* better PS core detection

* Add reference to UseCompatibleCmdlets doc

* changes a/p Chris

* Update RuleDocumentation/AvoidOverwritingBuiltInCmdlets.md

Co-Authored-By: Christoph Bergmeister [MVP] <c.bergmeister@gmail.com>

* trimmed doc and changed functiondefinitions detection to be more performant

* retrigger-ci after fix was made in master

* retrigger-ci due to sporadic test failure

* Update number of expected rules due to recent merge of PR #1373