Remove dotnet format (#16)

Author: emgarten

build.ps1

@@ -7,17 +7,14 @@ param (
 
 $RepoName = "NuGet.Test.Helpers"
 $RepoRoot = $PSScriptRoot
-pushd $RepoRoot
+Push-Location $RepoRoot
 
 # Load common build script helper methods
 . "$PSScriptRoot\build\common\common.ps1"
 
 # Download tools
 Install-CommonBuildTools $RepoRoot
 
-# Run dotnet-format to apply style fixes or fail on CI builds
-Invoke-DotnetFormat $RepoRoot
-
 # Clean and write git info
 Remove-Artifacts $RepoRoot
 Invoke-DotnetMSBuild $RepoRoot ("build\build.proj", "/t:Clean;WriteGitInfo", "/p:Configuration=$Configuration")
@@ -41,5 +38,5 @@ if (-not $SkipTests)
 # Run build.proj
 Invoke-DotnetMSBuild $RepoRoot ("build\build.proj", "/t:$buildTargets", "/p:Configuration=$Configuration")
 
-popd
+Pop-Location
 Write-Host "Success!"

build/common/common.ps1

@@ -126,53 +126,11 @@ Function Invoke-DotnetMSBuild {
     Invoke-DotnetExe $RepoRoot $buildArgs
 }
 
-Function Install-DotnetTools {
-    param(
-        [string]$RepoRoot
-    )
-
-    $toolsPath = Join-Path $RepoRoot ".nuget/tools"
-
-    if (-not (Test-Path $toolsPath)) {
-        Write-Host "Installing dotnet tools to $toolsPath"
-        $args = @("tool","install","--tool-path",$toolsPath,"--ignore-failed-sources","dotnet-format","--version","5.1.250801")
-
-        Invoke-DotnetExe $RepoRoot $args
-    }
-}
-
 Function Install-CommonBuildTools {
     param(
         [string]$RepoRoot
     )
 
     Install-DotnetCLI $RepoRoot
     Install-NuGetExe $RepoRoot
-    Install-DotnetTools $RepoRoot
-}
-
-Function Invoke-DotnetFormat {
-    param(
-        [string]$RepoRoot
-    )
-
-    # Only run in local dev envs
-    if ($env:CI -ne "True") 
-    {
-        $formatExe = Join-Path $RepoRoot ".nuget/tools/dotnet-format.exe"
-
-        $args = @("--fix-whitespace", "--fix-style", "warn")
-
-        $command = "$formatExe $args"
-        Write-Host "[EXEC] $command" -ForegroundColor Cyan
-
-        & $formatExe $args
-
-        if (-not $?) {
-            Write-Warning "dotnet-format failed. Please fix the style errors!"
-
-            # Currently dotnet-format fails on CIs but not locally in some scenarios
-            # exit 1
-        }
-    }
 }

build/common/common.sh

@@ -7,8 +7,6 @@ run_standard_tests()
   # Download dotnet cli
   REPO_ROOT=$(pwd)
   DOTNET=$(pwd)/.cli/dotnet
-  DOTNET_TOOLS=$(pwd)/.nuget/tools
-  DOTNET_FORMAT=$DOTNET_TOOLS/dotnet-format
 
   if [ ! -f $DOTNET ]; then
     echo "Installing dotnet"
@@ -24,16 +22,6 @@ run_standard_tests()
   # Display info
   $DOTNET --info
 
-  # install dotnet-format
-  if [ ! -d $DOTNET_TOOLS ]; then
-    echo "Installing dotnet tools"
-    mkdir -p .nuget/tools
-    
-    $DOTNET tool install --tool-path $DOTNET_TOOLS --ignore-failed-sources dotnet-format --version 5.1.250801
-  fi
-
-  $DOTNET_FORMAT --fix-whitespace --fix-style warn
-
   # clean
   rm -r -f $(pwd)/artifacts