Refactor Libraries directories to SDK format (#2436)

* Rename /libs to /libraries to match SDK
Rename libraries solution to Microsoft.Bot.Builder.Solutions
Extract test projects to /tests directories

* Fix .snk file references

* Update to:
\sdk
    \csharp
        \build
        \libraries
            \microsoft.bot.builder.skills
            \microsoft.bot.builder.solutions
        \tests
            \microsoft.bot.builder.skills.tests
            \microsoft.bot.builder.solutions.tests
        Microsoft.Bot.Builder.Solutions.sln

* Rename microsoft.bot.builder.skills to microsoft.bot.builder.solutions.skills
Refactor typescript libraries to follow
\sdk
    \typescript
        \libraries
            \botbuilder-skills
            \botbuilder-solutions

Move .ps1 files to \build and update references within
Update yaml files with new references (csharp and typescript)
Remove VirtualAssistant.ruleset at root

* Updated npm pack references from \build\outputpackages directory

* Fix reference to Skills project in solution
Rename Skills test folder to Microsoft.Bot.Builder.Solutions.Skills.Tests

* Match botbuilder.solutions.skills.yml to new directory

* Remove .snk file from /build directory as it is duplicated in /sdk/csharp/build

* Move \yaml as subdirectory under \build

* Fix ref to microsoft.bot.builder.solutions.skills directory

Author: ryanisgrig

VirtualAssistant.ruleset

@@ -6,7 +6,7 @@ if (-not $version) {
     Write-Host "Version for botbuilder-libs required!.  Please use the param -version" -ForegroundColor DarkRed
 }
 
-pushd .\lib\typescript
+pushd ..\sdk\typescript\libraries
 
 node .\common\scripts\install-run-rush.js install --no-link
 
@@ -35,7 +35,7 @@ if (-not(test-path ".\outputpackages"))
 
 pushd .\outputpackages
 
-npm pack ..\lib\typescript\botbuilder-solutions
-npm pack ..\lib\typescript\botbuilder-skills
+npm pack ..\..\sdk\typescript\libraries\botbuilder-solutions
+npm pack ..\..\sdk\typescript\libraries\botbuilder-skills
 
 popd

build.botbuilder-libs.ts.ps1 renamed to build/build.botbuilder-libs.ts.ps1

@@ -6,7 +6,7 @@ if (-not $version) {
     Write-Host "Version for botskills CLI tool required!.  Please use the param -version" -ForegroundColor DarkRed
 }
 
-pushd .\tools\botskills
+pushd ..\tools\botskills
 
 npm install
 npm version $($version) --allow-same-version
@@ -21,6 +21,6 @@ if (-not(test-path ".\outputpackages"))
 
 pushd .\outputpackages
 
-npm pack ..\tools\botskills
+npm pack ..\..\tools\botskills
 
 popd

build.botskills.ts.ps1 renamed to build/build.botskills.ts.ps1

@@ -6,7 +6,7 @@ if (-not $version) {
     Write-Host "Version for generator-botbuilder-assistant required!.  Please use the param -version" -ForegroundColor DarkRed
 }
 
-pushd .\templates\Virtual-Assistant-Template\typescript\generator-botbuilder-assistant
+pushd ..\templates\Virtual-Assistant-Template\typescript\generator-botbuilder-assistant
 
 npm install
 npm version $($version) --allow-same-version
@@ -21,6 +21,6 @@ if (-not(test-path ".\outputpackages"))
 
 pushd .\outputpackages
 
-npm pack ..\templates\Virtual-Assistant-Template\typescript\generator-botbuilder-assistant
+npm pack ..\..\templates\Virtual-Assistant-Template\typescript\generator-botbuilder-assistant
 
 popd

build.generator-assistant.ts.ps1 renamed to build/build.generator-assistant.ts.ps1

@@ -4,10 +4,10 @@ trigger:
     include:
     - master
     - feature/*
-  
+
   paths:
     include:
-    - 'lib/csharp/microsoft.bot.builder.solutions/*' 
+    - 'sdk/csharp/libraries/microsoft.bot.builder.solutions.skills/*'
 
 # By default will disable PR builds
 pr: none
@@ -37,12 +37,12 @@ steps:
 - task: NuGetCommand@2
   displayName: 'NuGet restore'
   inputs:
-    restoreSolution: 'lib\csharp\Microsoft.Bot.Builder.Skills.sln'
+    restoreSolution: 'sdk\csharp\Microsoft.Bot.Builder.Solutions.sln'
 
 - task: VSBuild@1
-  displayName: 'Build solution Microsoft.Bot.Builder.Skills.sln'
+  displayName: 'Build solution Microsoft.Bot.Builder.Solutions.sln'
   inputs:
-    solution: lib\csharp\Microsoft.Bot.Builder.Skills.sln
+    solution: sdk\csharp\Microsoft.Bot.Builder.Solutions.sln
     vsVersion: '16.0'
     platform: '$(buildPlatform)'
     configuration: '$(buildConfiguration)'
@@ -51,13 +51,13 @@ steps:
   displayName: 'test results'
   inputs:
     command: test
-    projects: '$(System.DefaultWorkingDirectory)\lib\csharp\microsoft.bot.builder.solutions\microsoft.bot.builder.solutions.tests\Microsoft.Bot.Builder.Solutions.Tests.csproj'
+    projects: '$(System.DefaultWorkingDirectory)\sdk\csharp\tests\microsoft.bot.builder.solutions.skills.tests\Microsoft.Bot.Builder.Solutions.Skills.Tests.csproj'
     arguments: '-v n --configuration $(buildConfiguration) --no-build --no-restore --filter TestCategory!=IgnoreInAutomatedBuild /p:CollectCoverage=true /p:CoverletOutputFormat=cobertura'
-    workingDirectory: 'lib\csharp\microsoft.bot.builder.solutions\microsoft.bot.builder.solutions.tests'
+    workingDirectory: 'sdk\csharp\tests\microsoft.bot.builder.solutions.skills.tests'
 
 - task: PublishCodeCoverageResults@1
   displayName: 'Publish code coverage'
   inputs:
     codeCoverageTool: Cobertura
-    summaryFileLocation: '$(Build.SourcesDirectory)\lib\csharp\microsoft.bot.builder.solutions\microsoft.bot.builder.solutions.tests\coverage.cobertura.xml'
-    reportDirectory: '$(Build.SourcesDirectory)\lib\csharp\microsoft.bot.builder.solutions\microsoft.bot.builder.solutions.tests'
+    summaryFileLocation: '$(Build.SourcesDirectory)\sdk\csharp\tests\microsoft.bot.builder.solutions.skills.tests\Microsoft.Bot.Builder.Solutions.Skills.Tests\coverage.cobertura.xml'
+    reportDirectory: '$(Build.SourcesDirectory)\sdk\csharp\tests\microsoft.bot.builder.solutions.skills.tests'

yaml/csharp/README.md renamed to build/yaml/csharp/README.md

@@ -4,10 +4,10 @@ trigger:
     include:
     - master
     - feature/*
-
+  
   paths:
     include:
-    - 'lib/csharp/microsoft.bot.builder.skills/*'
+    - 'sdk/csharp/libraries/microsoft.bot.builder.solutions/*' 
 
 # By default will disable PR builds
 pr: none
@@ -37,12 +37,12 @@ steps:
 - task: NuGetCommand@2
   displayName: 'NuGet restore'
   inputs:
-    restoreSolution: 'lib\csharp\Microsoft.Bot.Builder.Skills.sln'
+    restoreSolution: 'sdk\csharp\Microsoft.Bot.Builder.Solutions.sln'
 
 - task: VSBuild@1
-  displayName: 'Build solution Microsoft.Bot.Builder.Skills.sln'
+  displayName: 'Build solution Microsoft.Bot.Builder.Solutions.sln'
   inputs:
-    solution: lib\csharp\Microsoft.Bot.Builder.Skills.sln
+    solution: sdk\csharp\Microsoft.Bot.Builder.Solutions.sln
     vsVersion: '16.0'
     platform: '$(buildPlatform)'
     configuration: '$(buildConfiguration)'
@@ -51,13 +51,13 @@ steps:
   displayName: 'test results'
   inputs:
     command: test
-    projects: '$(System.DefaultWorkingDirectory)\lib\csharp\microsoft.bot.builder.skills\Microsoft.Bot.Builder.Skills.Tests\Microsoft.Bot.Builder.Skills.Tests.csproj'
+    projects: '$(System.DefaultWorkingDirectory)\sdk\csharp\tests\microsoft.bot.builder.solutions.tests\Microsoft.Bot.Builder.Solutions.Tests.csproj'
     arguments: '-v n --configuration $(buildConfiguration) --no-build --no-restore --filter TestCategory!=IgnoreInAutomatedBuild /p:CollectCoverage=true /p:CoverletOutputFormat=cobertura'
-    workingDirectory: 'lib\csharp\microsoft.bot.builder.skills\Microsoft.Bot.Builder.Skills.Tests'
+    workingDirectory: 'sdk\csharp\tests\microsoft.bot.builder.solutions.tests'
 
 - task: PublishCodeCoverageResults@1
   displayName: 'Publish code coverage'
   inputs:
     codeCoverageTool: Cobertura
-    summaryFileLocation: '$(Build.SourcesDirectory)\lib\csharp\microsoft.bot.builder.skills\Microsoft.Bot.Builder.Skills.Tests\coverage.cobertura.xml'
-    reportDirectory: '$(Build.SourcesDirectory)\lib\csharp\microsoft.bot.builder.skills\Microsoft.Bot.Builder.Skills.Tests'
+    summaryFileLocation: '$(Build.SourcesDirectory)\sdk\csharp\tests\microsoft.bot.builder.solutions.tests\coverage.cobertura.xml'
+    reportDirectory: '$(Build.SourcesDirectory)\sdk\csharp\tests\microsoft.bot.builder.solutions.tests'