|
1 | | -<html> |
2 | | -<head> |
3 | | - <title></title> |
4 | | -</head> |
5 | | -<body> |
6 | | -<p><strong>Introduction</strong></p> |
| 1 | +Introduction |
| 2 | +============ |
7 | 3 |
|
8 | | -<p>ScriptAnalyzer is a static code checker for Windows PowerShell modules and scripts. ScriptAnalyzer checks the quality of Windows PowerShell code by running a set of rules. The rules are based on PowerShell best practices identified by PowerShell Team and the community. It generates DiagnosticResults (errors and warnings) to inform users about potential code defects and suggests possible solutions for improvements.</p> |
| 4 | +ScriptAnalyzer is a static code checker for Windows PowerShell modules and scripts. ScriptAnalyzer checks the quality of Windows PowerShell code by running a set of rules. The rules are based on PowerShell best practices identified by PowerShell Team and the community. It generates DiagnosticResults (errors and warnings) to inform users about potential code defects and suggests possible solutions for improvements. |
9 | 5 |
|
10 | | -<p>ScriptAnalyzer is shipped with a collection of built-in rules that checks various aspects of PowerShell code such as presence of uninitialized variables, usage of PSCredential Type, usage of Invoke-Expression etc. Additional functionalities such as exclude/include specific rules are also supported.</p> |
| 6 | +ScriptAnalyzer is shipped with a collection of built-in rules that checks various aspects of PowerShell code such as presence of uninitialized variables, usage of PSCredential Type, usage of Invoke-Expression etc. Additional functionalities such as exclude/include specific rules are also supported. |
11 | 7 |
|
12 | | -<p> </p> |
| 8 | +ScriptAnalyzer cmdlets |
| 9 | +====================== |
| 10 | +``` |
| 11 | +Get-ScriptAnalyzerRule [-CustomizedRulePath <string[]>] [-Name <string[]>] [<CommonParameters>] |
13 | 12 |
|
14 | | -<p><strong>ScriptAnalyzer cmdlets:</strong></p> |
| 13 | +Invoke-ScriptAnalyzer [-Path] <string> [-CustomizedRulePath <string[]>] [-ExcludeRule <string[]>] [-IncludeRule <string[]>] [-Severity <string[]>] [-Recurse] [<CommonParameters>] |
| 14 | +``` |
15 | 15 |
|
16 | | -<p>Get-ScriptAnalyzerRule [-CustomizedRulePath <string[]>]<br /> |
17 | | - [-Name <string[]>]<br /> |
18 | | - [<CommonParameters>]</p> |
| 16 | +Requirements |
| 17 | +============ |
19 | 18 |
|
20 | | -<p><br /> |
21 | | -Invoke-ScriptAnalyzer [-Path] <string> [-CustomizedRulePath <string[]>] <br /> |
22 | | - [-ExcludeRule <string[]>] <br /> |
23 | | - [-IncludeRule <string[]>] <br /> |
24 | | - [-Severity <string[]>] <br /> |
25 | | - [-Recurse] <br /> |
26 | | - [<CommonParameters>]</p> |
| 19 | +WS2012R2 / Windows 8.1 / Windows OS running PowerShell v5.0 which can be obtained using [Windows Management Framework 5.0 Preview February 2015](http://go.microsoft.com/fwlink/?LinkId=398175). |
27 | 20 |
|
28 | | -<p> </p> |
| 21 | +Installation |
| 22 | +============ |
29 | 23 |
|
30 | | -<p> </p> |
| 24 | +1. Build the Code using Visual Studio |
31 | 25 |
|
32 | | -<p><strong>Requirements</strong></p> |
| 26 | +2. Copy following files to ```$env:ProgramFiles\WindowsPowerShell\Modules\ScriptAnalyzer``` |
33 | 27 |
|
34 | | -<p>WS2012R2 / Windows 8.1 / Windows OS containing PowerShell v5.0 which can be obtained using Windows Management Framework 5.0 Preview February 2015.</p> |
| 28 | +3. In PowerShell Console: |
| 29 | +```powershell |
| 30 | +Import-Module $env:ProgramFiles\WindowsPowerShell\Modules\ScriptAnalyzer\PSScriptAnalyzer.psd1 |
| 31 | +``` |
35 | 32 |
|
36 | | -<p> </p> |
| 33 | +To confirm installation: run ```Get-ScriptAnalyzerRule``` in the PowerShell console to obtain the built-in rules |
37 | 34 |
|
38 | | -<p><strong>Installation</strong></p> |
| 35 | +Building the Code |
| 36 | +================= |
39 | 37 |
|
40 | | -<p>1) Build the Code using Visual Studio</p> |
| 38 | +Use Visual Studio or any C# compiler to build the code |
41 | 39 |
|
42 | | -<p>2) Copy following files to $env:ProgramFiles\WindowsPowerShell\Modules\ScriptAnalyzer</p> |
| 40 | +Running Tests |
| 41 | +============= |
43 | 42 |
|
44 | | -<p>3) In PowerShell Console :<br /> |
45 | | -Import-Module $env:ProgramFiles\WindowsPowerShell\Modules\ScriptAnalyzer\PSScriptAnalyzer.psd1</p> |
| 43 | +Pester-based ScriptAnalyzer Tests are located in ```<branch>/ScriptAnalyzer/Tests``` folder |
46 | 44 |
|
47 | | -<p>To confirm installation:</p> |
| 45 | +* Ensure Pester is installed on the machine |
| 46 | +* Go the Tests folder in your local repository |
| 47 | +* Run Engine Tests: |
| 48 | +.\InvokeScriptAnalyzer.tests.ps1 |
| 49 | +* Run Tests for Built-in rules: |
| 50 | +.\*.ps1 (Example - .\ AvoidConvertToSecureStringWithPlainText.ps1) |
| 51 | + |
48 | 52 |
|
49 | | -<p>· Run Get-ScriptAnalyzerRule in the PowerShell console to obtain the built-in rules</p> |
| 53 | +Contributing to ScriptAnalyzer |
| 54 | +============================== |
50 | 55 |
|
51 | | -<p> </p> |
| 56 | +You are welcome to contribute to this project. There are many ways to contribute: |
52 | 57 |
|
53 | | -<p><br /> |
54 | | -<strong>Building the Code</strong></p> |
| 58 | +1. Submit a bug report via [Issues]( https://github.com/PowerShell/ScriptAnalyzer/issues). For a guide to submitting good bug reports, please read [Painless Bug Tracking](http://www.joelonsoftware.com/articles/fog0000000029.html). |
| 59 | +2. Verify fixes for bugs. |
| 60 | +3. Submit your fixes for a bug. Before submitting, please make sure you have: |
| 61 | + * Performed code reviews of your own |
| 62 | + * Updated the test cases if needed |
| 63 | + * Run the test cases to ensure no feature breaks or test breaks |
| 64 | + * Added the test cases for new code |
| 65 | +4. Submit a feature request. |
| 66 | +5. Help answer questions in the discussions list. |
| 67 | +6. Submit test cases. |
| 68 | +7. Tell others about the project. |
| 69 | +8. Tell the developers how much you appreciate the product! |
55 | 70 |
|
56 | | -<p>Use Visual Studio or any C# compiler to build the code</p> |
| 71 | +You might also read these two blog posts about contributing code: [Open Source Contribution Etiquette](http://tirania.org/blog/archive/2010/Dec-31.html) by Miguel de Icaza, and [Don’t “Push” Your Pull Requests](http://www.igvita.com/2011/12/19/dont-push-your-pull-requests/) by Ilya Grigorik. |
57 | 72 |
|
58 | | -<p> </p> |
59 | | - |
60 | | -<p><br /> |
61 | | -<strong>Running Tests</strong></p> |
62 | | - |
63 | | -<p>Pester based ScriptAnalyzer Tests are located in “<branch>/ScriptAnalyzer/Tests” folder</p> |
64 | | - |
65 | | -<ul> |
66 | | - <li>Ensure Pester is installed on the machine</li> |
67 | | - <li>Go the Tests folder in your local repository</li> |
68 | | - <li>Run Engine Tests: |
69 | | - <ul> |
70 | | - <li>.\InvokeScriptAnalyzer.tests.ps1</li> |
71 | | - </ul> |
72 | | - </li> |
73 | | - <li>Run Tests for Built-in rules: |
74 | | - <ul> |
75 | | - <li>.\*.ps1 (Example - .\ AvoidConvertToSecureStringWithPlainText.ps1)<br /> |
76 | | - </li> |
77 | | - </ul> |
78 | | - </li> |
79 | | -</ul> |
80 | | - |
81 | | -<p><br /> |
82 | | -<strong>Contributing to ScriptAnalyzer</strong></p> |
83 | | - |
84 | | -<p>You are welcome to contribute to this project. There are many ways to contribute:</p> |
85 | | - |
86 | | -<ul> |
87 | | - <li>Submit a bug report via Issues. For a guide to submitting good bug reports, please read Painless Bug Tracking.</li> |
88 | | - <li>Verify fixes for bugs.</li> |
89 | | - <li>Submit your fixes for a bug. Before submitting, please make sure you have: |
90 | | - <ul> |
91 | | - <li>Performed code reviews of your own</li> |
92 | | - <li>Updated the test cases if needed</li> |
93 | | - <li>Run the test cases to ensure no feature breaks or test breaks</li> |
94 | | - <li>Added the test cases for new code</li> |
95 | | - </ul> |
96 | | - </li> |
97 | | - <li>Submit a feature request.</li> |
98 | | - <li>Help answer questions in the discussions list.</li> |
99 | | - <li>Submit test cases.</li> |
100 | | - <li>Tell others about the project.</li> |
101 | | - <li>Tell the developers how much you appreciate the product!</li> |
102 | | -</ul> |
103 | | - |
104 | | -<p><br /> |
105 | | -You might also read these two blog posts about contributing code: Open Source Contribution Etiquette by Miguel de Icaza, and Don’t “Push” Your Pull Requests by Ilya Grigorik.</p> |
106 | | - |
107 | | -<p>Before submitting a feature or substantial code contribution, please discuss it with the Windows PowerShell team via Issues, and ensure it follows the product roadmap. Note that all code submissions will be rigorously reviewed by the Windows PowerShell Team. Only those that meet a high bar for both quality and roadmap fit will be merged into the source.</p> |
108 | | - |
109 | | -<p> </p> |
110 | | -</body> |
111 | | -</html> |
| 73 | +Before submitting a feature or substantial code contribution, please discuss it with the Windows PowerShell team via [Issues](https://github.com/PowerShell/ScriptAnalyzer/issues), and ensure it follows the product roadmap. Note that all code submissions will be rigorously reviewed by the Windows PowerShell Team. Only those that meet a high bar for both quality and roadmap fit will be merged into the source. |
0 commit comments