Merge remote-tracking branch 'upstream/dev16.0.x' into bring-ranges-u…

…p-to-date

.editorconfig

@@ -47,6 +47,75 @@ dotnet_style_coalesce_expression = true:suggestion
 dotnet_style_null_propagation = true:suggestion
 dotnet_style_explicit_tuple_names = true:suggestion
 
+# Non-private static fields are PascalCase
+dotnet_naming_rule.non_private_static_fields_should_be_pascal_case.severity = suggestion
+dotnet_naming_rule.non_private_static_fields_should_be_pascal_case.symbols = non_private_static_fields
+dotnet_naming_rule.non_private_static_fields_should_be_pascal_case.style = non_private_static_field_style
+
+dotnet_naming_symbols.non_private_static_fields.applicable_kinds = field
+dotnet_naming_symbols.non_private_static_fields.applicable_accessibilities = public, protected, internal, protected internal, private protected
+dotnet_naming_symbols.non_private_static_fields.required_modifiers = static
+
+dotnet_naming_style.non_private_static_field_style.capitalization = pascal_case
+
+# Constants are PascalCase
+dotnet_naming_rule.constants_should_be_pascal_case.severity = suggestion
+dotnet_naming_rule.constants_should_be_pascal_case.symbols = constants
+dotnet_naming_rule.constants_should_be_pascal_case.style = constant_style
+
+dotnet_naming_symbols.constants.applicable_kinds = field, local
+dotnet_naming_symbols.constants.required_modifiers = const
+
+dotnet_naming_style.constant_style.capitalization = pascal_case
+
+# Static fields are camelCase and start with s_
+dotnet_naming_rule.static_fields_should_be_camel_case.severity = suggestion
+dotnet_naming_rule.static_fields_should_be_camel_case.symbols = static_fields
+dotnet_naming_rule.static_fields_should_be_camel_case.style = static_field_style
+
+dotnet_naming_symbols.static_fields.applicable_kinds = field
+dotnet_naming_symbols.static_fields.required_modifiers = static
+
+dotnet_naming_style.static_field_style.capitalization = camel_case
+dotnet_naming_style.static_field_style.required_prefix = s_
+
+# Instance fields are camelCase and start with _
+dotnet_naming_rule.instance_fields_should_be_camel_case.severity = suggestion
+dotnet_naming_rule.instance_fields_should_be_camel_case.symbols = instance_fields
+dotnet_naming_rule.instance_fields_should_be_camel_case.style = instance_field_style
+
+dotnet_naming_symbols.instance_fields.applicable_kinds = field
+
+dotnet_naming_style.instance_field_style.capitalization = camel_case
+dotnet_naming_style.instance_field_style.required_prefix = _
+
+# Locals and parameters are camelCase
+dotnet_naming_rule.locals_should_be_camel_case.severity = suggestion
+dotnet_naming_rule.locals_should_be_camel_case.symbols = locals_and_parameters
+dotnet_naming_rule.locals_should_be_camel_case.style = camel_case_style
+
+dotnet_naming_symbols.locals_and_parameters.applicable_kinds = parameter, local
+
+dotnet_naming_style.camel_case_style.capitalization = camel_case
+
+# Local functions are PascalCase
+dotnet_naming_rule.local_functions_should_be_pascal_case.severity = suggestion
+dotnet_naming_rule.local_functions_should_be_pascal_case.symbols = local_functions
+dotnet_naming_rule.local_functions_should_be_pascal_case.style = local_function_style
+
+dotnet_naming_symbols.local_functions.applicable_kinds = local_function
+
+dotnet_naming_style.local_function_style.capitalization = pascal_case
+
+# By default, name items with PascalCase
+dotnet_naming_rule.members_should_be_pascal_case.severity = suggestion
+dotnet_naming_rule.members_should_be_pascal_case.symbols = all_members
+dotnet_naming_rule.members_should_be_pascal_case.style = pascal_case_style
+
+dotnet_naming_symbols.all_members.applicable_kinds = *
+
+dotnet_naming_style.pascal_case_style.capitalization = pascal_case
+
 # CSharp code style settings:
 [*.cs]
 # Indentation preferences

.gitattributes

@@ -2,16 +2,6 @@
 # Set default behavior to automatically normalize line endings.
 ###############################################################################
 * text=auto encoding=UTF-8
-
-# RunCsc/RunVbc are shell scripts and should always have unix line endings
-# These shell scripts are included in the toolset packages. Normally, the shell
-# scripts in our repo are only run by cloning onto a Linux/Mac machine, and git
-# automatically chooses LF as the line ending.
-#
-# However, right now the toolset packages must be built on Windows, and so the
-# files must be hard-coded to be cloned with LF
-build/NuGetAdditionalFiles/RunCsc text eol=lf
-build/NuGetAdditionalFiles/RunVbc text eol=lf
 *.sh text eol=lf
 
 ###############################################################################

.vsts-ci.yml

@@ -44,7 +44,7 @@ steps:
         /p:TreatWarningsAsErrors=true 
         /p:DeployExtension=false 
         /p:TrackFileAccess=false
-        /p:OfficialBuild=true 
+        /p:OfficialBuildId=$(BUILD.BUILDNUMBER)
         /p:VisualStudioVersion=14.0 
         /flp1:Summary;Verbosity=diagnostic;Encoding=UTF-8;LogFile=$(Build.SourcesDirectory)\Binaries\$(BuildConfiguration)\Roslyn.log 
         /flp2:WarningsOnly;Verbosity=diagnostic;Encoding=UTF-8;LogFile=$(Build.SourcesDirectory)\Binaries\$(BuildConfiguration)\Roslyn.wrn 
@@ -57,12 +57,15 @@ steps:
         /p:PB_PublishBlobFeedKey=$(PB_PublishBlobFeedKey) 
         /p:PublishStableVersions=false 
         /p:VersionStampToPublish=prerelease
+        /p:DotNetSymbolServerTokenMsdl=$(microsoft-symbol-server-pat)
+        /p:DotNetSymbolServerTokenSymWeb=$(symweb-symbol-server-pat)
     platform: '$(BuildPlatform)'
     configuration: '$(BuildConfiguration)'
     maximumCpuCount: true
     logProjectEvents: false
 
 - task: PublishBuildArtifacts@1
+  displayName: Publish Logs
   inputs:
     PathtoPublish: '$(Build.SourcesDirectory)\Binaries\$(BuildConfiguration)\Logs'
     ArtifactName: 'Build Diagnostic Files'
@@ -71,47 +74,18 @@ steps:
   condition: succeededOrFailed()
 
 - task: PublishTestResults@1
+  displayName: Publish Test Results
   inputs:
     testRunner: XUnit
     testResultsFiles: '**/xUnitResults/*.xml'
     mergeTestResults: true
     testRunTitle: 'Unit Tests'
   condition: succeededOrFailed()
 
-- task: PublishSymbols@1
-  displayName: Create Symbol Index
-  inputs:
-    SymbolsPath: '$(DropRoot)\Roslyn-Signed\$(Build.SourceBranchName)\$(BuildConfiguration)\$(Build.BuildNumber)\Symbols'
-    SearchPattern: '**\*.dll;**\*.exe;**\*.pdb'
-    SymbolsFolder: '$(Build.SourcesDirectory)\Binaries\$(BuildConfiguration)\SymStore'
-    SkipIndexing: true
-    TreatNotIndexedAsWarning: true
-    SymbolsProduct: Roslyn
-    SymbolsVersion: '$(Build.BuildNumber)'
-    SymbolsArtifactName: Symbols
-  condition: and(succeeded(), contains(variables['PB_PublishType'], 'vsts'))
-
-- task: CopyFiles@2
-  displayName: Copy Files to Local Temporary Path
-  inputs:
-    SourceFolder: '$(DropRoot)\Roslyn-Signed\$(Build.SourceBranchName)\$(BuildConfiguration)\$(Build.BuildNumber)\Symbols'
-    TargetFolder: '$(Build.StagingDirectory)\SymbolsForPublish'
-  condition: and(succeeded(), contains(variables['PB_PublishType'], 'vsts'))
-
-- task: ms-vscs-artifact.build-tasks.artifactSymbolTask-1.artifactSymbolTask@0
-  displayName: Publish Symbols to Artifact Services
-  inputs:
-    symbolServiceURI: 'https://microsoft.artifacts.visualstudio.com/DefaultCollection'
-    requestName: '$(system.teamProject)/Roslyn-Signed/$(Build.BuildNumber)/$(Build.BuildId)'
-    sourcePath: '$(Build.StagingDirectory)\SymbolsForPublish'
-    usePat: false
-  continueOnError: true
-  condition: and(succeeded(), contains(variables['PB_PublishType'], 'vsts'))
-
 - task: ms-vseng.MicroBuildTasks.4305a8de-ba66-4d8b-b2d1-0dc4ecbbf5e8.MicroBuildUploadVstsDropFolder@1
   displayName: Upload VSTS Drop
   inputs:
-    DropFolder: 'Binaries\$(BuildConfiguration)\Insertion'
+    DropFolder: 'Binaries\VSSetup\$(BuildConfiguration)\Insertion'
   condition: and(succeeded(), contains(variables['PB_PublishType'], 'vsts'))
 
 - task: NuGetCommand@2
@@ -124,15 +98,26 @@ steps:
     allowPackageConflicts: true
   condition: and(succeeded(), contains(variables['PB_PublishType'], 'vsts'))
 
+# Publish VSTS artifact that the RoslynInsertionTool is able to find by its name.
+# The backing storage of the artifact is a file share.
 - task: PublishBuildArtifacts@1
-  displayName: Publish Drop (parallel)
+  displayName: Publish Drop - Binaries
   inputs:
     PathtoPublish: 'Binaries\$(BuildConfiguration)'
     ArtifactName: '$(Build.BuildNumber)'
     publishLocation: FilePath
     TargetPath: '$(DropRoot)\Roslyn-Signed\$(Build.SourceBranchName)\$(BuildConfiguration)'
     Parallel: true
     ParallelCount: 64
+  condition: succeededOrFailed()
+
+# Copy additional files to the drop directory
+- task: PowerShell@2
+  displayName: Publish Drop - VSSetup
+  inputs:
+    arguments: '-Source Binaries\VSSetup\$(BuildConfiguration)\Insertion -Target $(DropRoot)\Roslyn-Signed\$(Build.SourceBranchName)\$(BuildConfiguration)\$(Build.BuildNumber)\Insertion -ParallelCount 8'
+    filePath: 'build\scripts\vsts-robocopy.ps1'
+  condition: succeededOrFailed()
 
 - task: ms-vseng.MicroBuildTasks.521a94ea-9e68-468a-8167-6dcf361ea776.MicroBuildCleanup@1
   displayName: Perform Cleanup Tasks

.vsts-dnceng.yml

@@ -0,0 +1,129 @@
+phases:
+- phase: Windows_Desktop_Unit_Tests
+  queue:
+    name: Helix
+    timeoutInMinutes: 90
+    parallel: 4
+    matrix:
+      debug_32:
+        _configuration: Debug
+        _testKind: Test32
+      debug_64:
+        _configuration: Debug
+        _testKind: Test64
+      release_32:
+        _configuration: Release
+        _testKind: Test32
+      release_64:
+        _configuration: Release
+        _testKind: Test64
+
+  steps:
+    - script: build/scripts/cibuild.cmd -$(_configuration) -testDesktop -$(_testKind)
+
+    - task: PublishTestResults@1
+      inputs:
+        testRunner: XUnit
+        testResultsFiles: '**/xUnitResults/*.xml'
+        mergeTestResults: true
+        testRunTitle: 'Windows Desktop $(_configuration) $(_testKind)'
+      condition: succeededOrFailed()
+
+    - task: PublishBuildArtifacts@1
+      inputs:
+        PathtoPublish: '$(Build.SourcesDirectory)\Binaries\$(_configuration)\Logs'
+        ArtifactName: 'Windows Desktop $(_configuration) $(_testKind)' 
+        publishLocation: Container
+      continueOnError: true
+      condition: failed()
+
+- phase: Windows_CoreClr_Unit_Tests
+  queue:
+    name: Helix
+    timeoutInMinutes: 90
+    parallel: 2
+    matrix:
+      debug:
+        _configuration: Debug
+      release:
+        _configuration: Release
+
+  steps:
+    - script: build/scripts/cibuild.cmd -$(_configuration) -testCoreClr -buildCoreClr
+
+    - task: PublishTestResults@1
+      inputs:
+        testRunner: XUnit
+        testResultsFiles: '**/xUnitResults/*.xml'
+        mergeTestResults: true
+        testRunTitle: 'Windows CoreClr $(_configuration)'
+      condition: succeededOrFailed()
+
+    - task: PublishBuildArtifacts@1
+      inputs:
+        PathtoPublish: '$(Build.SourcesDirectory)\Binaries\$(_configuration)\Logs'
+        ArtifactName: 'Windows CoreClr $(_configuration)'
+        publishLocation: Container
+      continueOnError: true
+      condition: failed()
+
+- phase: Windows_Determinism_Test
+  queue:
+    name: Helix
+    timeoutInMinutes: 90
+  steps:
+    - script: build/scripts/cibuild.cmd -testDeterminism
+
+    - task: PublishBuildArtifacts@1
+      inputs:
+        PathtoPublish: '$(Build.SourcesDirectory)\Binaries\Debug\Logs'
+        ArtifactName: 'Build Determinism Files'
+        publishLocation: Container
+      continueOnError: true
+      condition: failed()
+
+- phase: Windows_Correctness_Test
+  queue:
+    name: Helix
+    timeoutInMinutes: 90
+  steps:
+    - script: build/scripts/test-build-correctness.cmd -cibuild -release
+
+    - task: PublishBuildArtifacts@1
+      inputs:
+        PathtoPublish: '$(Build.SourcesDirectory)\Binaries\Release\Logs'
+        ArtifactName: 'Build Correctness Files'
+        publishLocation: Container
+      continueOnError: true
+      condition: failed()
+
+- phase: Linux_Test
+  queue:
+    name: DotNetCore-Linux                     
+    timeoutInMinutes: 90
+    parallel: 2
+    matrix:
+      coreclr:
+        _args: --debug
+        _name: CoreClr 
+      mono:
+        _args: --debug --docker --mono
+        _name: Mono
+  steps:
+    - script: ./build/scripts/cibuild.sh $(_args)
+    - task: PublishTestResults@1
+      inputs:
+        testRunner: XUnit
+        testResultsFiles: '**/xUnitResults/*.xml'
+        mergeTestResults: true
+        testRunTitle: 'Linux $(_name)'
+      condition: succeededOrFailed()
+
+    - task: PublishBuildArtifacts@1
+      inputs:
+        PathtoPublish: '$(Build.SourcesDirectory)/Binaries/$(_configuration)/Logs'
+        ArtifactName: 'Linux $(_name)'
+        publishLocation: Container
+      continueOnError: true
+      condition: failed()
+

Build.cmd

@@ -1,2 +1,2 @@
 @echo off
-powershell -noprofile -executionPolicy RemoteSigned -file "%~dp0\build\scripts\build.ps1" -build -skipBuildExtras %*
+powershell -noprofile -executionPolicy RemoteSigned -file "%~dp0\build\scripts\build.ps1" -build %*