Skip to content

Commit 7a82aa5

Browse files
committed
Merge branch 'preview' of https://github.com/Azure/azure-powershell into june
# Conflicts: # src/ResourceManager/Compute/Commands.Compute.Test/ScenarioTests/AEMExtensionTests.cs
2 parents 4536df6 + e78e977 commit 7a82aa5

File tree

209 files changed

+12933
-10640
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

209 files changed

+12933
-10640
lines changed

ChangeLog.md

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,8 @@
1-
## 6.2.0 - June 2018
1+
## 6.2.1 - June 2018
2+
### AzureRM.OperationalInsights
3+
* Updated PSWorkspace model to allow Network to use type as a parameter
4+
5+
## 6.2.0 - June 2018
26
#### AzureRM.Profile
37
* Fix issue where version 10.0.3 of Newtonsoft.Json wasn't being loaded on module import
48

build.proj

Lines changed: 30 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -68,7 +68,7 @@
6868
Condition=" '$(Scope)' == 'AzureStorage' "/>
6969
<CmdletSolutionsToBuild Include=".\src\Stack.sln"
7070
Condition=" '$(Stack)' == 'true' "/>
71-
71+
7272
<StaticAnalysis Include=".\tools\StaticAnalysis\StaticAnalysis.sln" />
7373
<LocalBuildTasks Include="$(LibraryToolsFolder)\BuildPackagesTask\Microsoft.Azure.Build.Tasks.sln" />
7474
<LocalBuildTasks Include="$(LibraryToolsFolder)\RepoTasks\RepoTasks.sln" Condition="'$(NetCore)' == 'false'"/>
@@ -204,26 +204,28 @@
204204
</FilesChangedTask>
205205

206206
<!-- Get the list of modules changed -->
207-
<FilterTask FilesChanged="@(FilesChanged)" MapFilePath=".\ModuleMappings.json">
208-
<Output TaskParameter="Output" ItemName="ModulesChanged"/>
209-
</FilterTask>
210-
<Message Text="Filtering help generation and StaticAnalysis by the following modules:"/>
211-
<Message Text="%(ModulesChanged.Identity)"/>
212-
<Message Text="Total: @(ModulesChanged->Count())"/>
213-
<Message Text=""/>
214-
215-
<!--Get the list of tests to be run based on files changed from a specified PullRequestNumber. Mapping between paths and test DLLs is used to produce the list.-->
216-
<FilterTask FilesChanged="@(FilesChanged)" MapFilePath=".\TestMappings.json">
217-
<Output TaskParameter="Output" ItemName="XUnitTests"/>
218-
</FilterTask>
219-
<Message Text="Using these test assemblies:"/>
220-
<Message Text="%(XUnitTests.Identity)"/>
221-
<Message Text="Total: @(XunitTests->Count())"/>
222-
<Message Text=""/>
207+
<FilterTask FilesChanged="@(FilesChanged)" MapFilePath=".\ModuleMappings.json">
208+
<Output TaskParameter="Output" ItemName="ModulesChanged"/>
209+
</FilterTask>
210+
<Message Text="Filtering help generation and StaticAnalysis by the following modules:"/>
211+
<Message Text="%(ModulesChanged.Identity)"/>
212+
<Message Text="Total: @(ModulesChanged->Count())"/>
213+
<Message Text=""/>
214+
215+
<!--Get the list of tests to be run based on files changed from a specified PullRequestNumber. Mapping between paths and test DLLs is used to produce the list.-->
216+
<FilterTask FilesChanged="@(FilesChanged)" MapFilePath=".\TestMappings.json">
217+
<Output TaskParameter="Output" ItemName="XUnitTests"/>
218+
</FilterTask>
219+
<Message Text="Using these test assemblies:"/>
220+
<Message Text="%(XUnitTests.Identity)"/>
221+
<Message Text="Total: @(XunitTests->Count())"/>
222+
<Message Text=""/>
223223
</Target>
224224

225225
<!-- Build all flavors of the Cmdlets -->
226226
<Target Name="Build" DependsOnTargets="RestoreNugetPackages;BuildMsBuildTask;FilterBuild">
227+
<CallTarget targets="ChangeLogCheck" ContinueOnError="false" />
228+
227229
<Message Importance="high" Text="Building Cmdlets..." />
228230

229231
<MakeDir Directories="$(PackageDirectory)" />
@@ -506,6 +508,17 @@
506508
<Error Text="StaticAnalysis has failed. Please follow the instructions on this doc: https://github.com/Azure/azure-powershell/blob/preview/documentation/Debugging-StaticAnalysis-Errors.md"/>
507509
</Target>
508510

511+
<Target Name="ChangeLogCheck">
512+
<Exec Command="&quot;$(PowerShellCommand)&quot; -NonInteractive -NoLogo -NoProfile -Command &quot;. $(LibraryToolsFolder)\CheckChangeLog.ps1 -FilesChanged '@(FilesChanged)' &quot;"
513+
ContinueOnError="false"
514+
Condition="'$(Latest)' == 'true'"/>
515+
<OnError ExecuteTargets="ChangeLogErrorMessage"/>
516+
</Target>
517+
518+
<Target Name="ChangeLogErrorMessage">
519+
<Error Text="Modified files were found with no update to their change log. Please add a snippet to the affected modules' change log."/>
520+
</Target>
521+
509522
<!-- Publish all packages -->
510523
<Target Name="Publish">
511524
<Error Condition=" '$(NuGetKey)' == '' " Text="You must provide the NuGetKey parameter to the build: /p:NuGetKey=YOUR_PUBLISHING_KEY" />

documentation/development-docs/azure-powershell-developer-guide.md

Lines changed: 4 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -40,7 +40,7 @@ The Azure PowerShell Developer Guide was created to help with the development an
4040
- [After Development](#after-development)
4141
- [Misc](#misc)
4242
- [Publish to PowerShell Gallery](#publish-to-powershell-gallery)
43-
- [AsJob Parameters](#asjob-parameters)
43+
- [AsJob Parameter](#asjob-parameter)
4444
- [Argument Completers](#argument-completers)
4545
- [Resource Group Completer](#resource-group-completers)
4646
- [Location Completer](#location-completer)
@@ -52,8 +52,6 @@ The following prerequisites should be completed before contributing to the Azure
5252

5353
- Install [Visual Studio 2015](https://www.visualstudio.com/downloads/)
5454
- Install the latest version of [Git](https://git-scm.com/downloads)
55-
- Install the latest version of [WiX](http://wixtoolset.org/releases/)
56-
- After installation, ensure that the path to "WiX Toolset\bin" has been added to your `PATH` environment variable
5755
- Install the [`platyPS` module](https://github.com/Azure/azure-powershell/blob/preview/documentation/development-docs/help-generation.md#installing-platyps)
5856
- Set the PowerShell [execution policy](https://technet.microsoft.com/en-us/library/ee176961.aspx) to **Unrestricted** for the following versions of PowerShell:
5957
- `C:\Windows\System32\WindowsPowerShell\v1.0\powershell.exe`
@@ -116,8 +114,6 @@ By default, we build the `dll-Help.xml` files (used to display the help content
116114
msbuild build.proj /p:SkipHelp=true
117115
```
118116

119-
_Note_: when [updating the installer](#updating-the-installer), you **should not** skip the help generation step, as it removes the `dll-Help.xml` files from the wxi file.
120-
121117
## Running Tests
122118

123119
With the same terminal open from the previous section, run the cmdlet `Invoke-CheckinTests` to run all of the tests in the project
@@ -142,10 +138,9 @@ Before development, you must meet with the Azure PowerShell team to have a desig
142138

143139
Before submitting a design review, please be sure that you have read the [Azure PowerShell Design Guidelines](./azure-powershell-design-guidelines.md) document.
144140

145-
Please email the **azdevxpsdr** alias to set up this review and include the following information:
146-
- Short description of the top-level scenarios
147-
- Proposed cmdlet syntax
148-
- Sample output of cmdlets
141+
Please submit a design review here: https://github.com/Azure/azure-powershell-cmdlet-review-pr
142+
143+
_Note_: You will need to be part of the GitHub Azure org to see this repository. Please go to [this link](aka.ms/azuregithub) to become part of the Azure org.
149144

150145
We recommend using the `platyPS` module to easily generate markdown files that contains the above information and including the files in the email.
151146

0 commit comments

Comments
 (0)