diff --git a/.appveyor.yml b/.appveyor.yml index 877eb1c..9a8e52c 100644 --- a/.appveyor.yml +++ b/.appveyor.yml @@ -1,7 +1,7 @@ #---------------------------------# # Build Image # #---------------------------------# -image: Visual Studio 2019 +image: Visual Studio 2022 #---------------------------------# # Build Script # @@ -11,7 +11,7 @@ install: - ps: nuget update -self build_script: - - ps: .\build.ps1 -Target AppVeyor + - ps: .\build.ps1 --target=CI # Tests test: off diff --git a/.config/dotnet-tools.json b/.config/dotnet-tools.json new file mode 100644 index 0000000..4903ba5 --- /dev/null +++ b/.config/dotnet-tools.json @@ -0,0 +1,12 @@ +{ + "version": 1, + "isRoot": true, + "tools": { + "cake.tool": { + "version": "1.3.0", + "commands": [ + "dotnet-cake" + ] + } + } +} \ No newline at end of file diff --git a/.github/dependabot.yml b/.github/dependabot.yml index 2d7664a..409c718 100644 --- a/.github/dependabot.yml +++ b/.github/dependabot.yml @@ -9,41 +9,13 @@ updates: ignore: - dependency-name: Cake.Core versions: - - ">= 0.36.a, < 0.37" - - dependency-name: Cake.Core - versions: - - ">= 0.37.a, < 0.38" - - dependency-name: Cake.Core - versions: - - ">= 0.38.a, < 0.39" - - dependency-name: Cake.Issues - versions: - - "> 0.9.0, < 0.10" - - dependency-name: Cake.Testing - versions: - - ">= 0.36.a, < 0.37" + - "(,3.0)" - dependency-name: Cake.Testing versions: - - ">= 0.37.a, < 0.38" - - dependency-name: Cake.Testing - versions: - - ">= 0.38.a, < 0.39" - - dependency-name: Microsoft.NET.Test.Sdk - versions: - - 16.8.3 - - 16.9.1 - - dependency-name: Cake.Core - versions: - - 1.0.0 - - dependency-name: Cake.Testing - versions: - - 1.0.0 - - dependency-name: Shouldly - versions: - - 4.0.3 - - dependency-name: Microsoft.CodeAnalysis.FxCopAnalyzers + - "(,3.0)" + - dependency-name: Cake.Issues versions: - - 3.3.2 + - "> 1.0.0, < 2" - dependency-name: Cake.Issues.Testing versions: - - 0.9.1 + - "> 1.0.0, < 2" diff --git a/azure-pipelines.yml b/azure-pipelines.yml index d7198be..c2b96e3 100644 --- a/azure-pipelines.yml +++ b/azure-pipelines.yml @@ -12,7 +12,7 @@ pr: jobs: - job: Windows pool: - vmImage: 'vs2017-win2016' + vmImage: 'windows-2022' steps: - powershell: ./build.ps1 displayName: 'Cake Build' \ No newline at end of file diff --git a/build.ps1 b/build.ps1 index faee428..0940a31 100644 --- a/build.ps1 +++ b/build.ps1 @@ -1,235 +1,15 @@ -########################################################################## -# This is the Cake bootstrapper script for PowerShell. -# This file was downloaded from https://github.com/cake-build/resources -# Feel free to change this file to fit your needs. -########################################################################## +$ErrorActionPreference = 'Stop' -<# +$SCRIPT_NAME = "recipe.cake" -.SYNOPSIS -This is a Powershell script to bootstrap a Cake build. +Write-Host "Restoring .NET Core tools" +dotnet tool restore +if ($LASTEXITCODE -ne 0) { exit $LASTEXITCODE } -.DESCRIPTION -This Powershell script will download NuGet if missing, restore NuGet tools (including Cake) -and execute your Cake build script with the parameters you provide. +Write-Host "Bootstrapping Cake" +dotnet cake $SCRIPT_NAME --bootstrap +if ($LASTEXITCODE -ne 0) { exit $LASTEXITCODE } -.PARAMETER Script -The build script to execute. -.PARAMETER Target -The build script target to run. -.PARAMETER Configuration -The build configuration to use. -.PARAMETER Verbosity -Specifies the amount of information to be displayed. -.PARAMETER ShowDescription -Shows description about tasks. -.PARAMETER DryRun -Performs a dry run. -.PARAMETER Experimental -Uses the nightly builds of the Roslyn script engine. -.PARAMETER Mono -Uses the Mono Compiler rather than the Roslyn script engine. -.PARAMETER SkipToolPackageRestore -Skips restoring of packages. -.PARAMETER ScriptArgs -Remaining arguments are added here. - -.LINK -https://cakebuild.net - -#> - -[CmdletBinding()] -Param( - [string]$Script = "recipe.cake", - [string]$Target = "Default", - [ValidateSet("Release", "Debug")] - [string]$Configuration = "Release", - [ValidateSet("Quiet", "Minimal", "Normal", "Verbose", "Diagnostic")] - [string]$Verbosity = "Verbose", - [switch]$ShowDescription, - [Alias("WhatIf", "Noop")] - [switch]$DryRun, - [switch]$Experimental, - [switch]$Mono, - [switch]$SkipToolPackageRestore, - [Parameter(Position=0,Mandatory=$false,ValueFromRemainingArguments=$true)] - [string[]]$ScriptArgs -) - -[Reflection.Assembly]::LoadWithPartialName("System.Security") | Out-Null -function MD5HashFile([string] $filePath) -{ - if ([string]::IsNullOrEmpty($filePath) -or !(Test-Path $filePath -PathType Leaf)) - { - return $null - } - - [System.IO.Stream] $file = $null; - [System.Security.Cryptography.MD5] $md5 = $null; - try - { - $md5 = [System.Security.Cryptography.MD5]::Create() - $file = [System.IO.File]::OpenRead($filePath) - return [System.BitConverter]::ToString($md5.ComputeHash($file)) - } - finally - { - if ($file -ne $null) - { - $file.Dispose() - } - } -} - -function GetProxyEnabledWebClient -{ - $wc = New-Object System.Net.WebClient - $proxy = [System.Net.WebRequest]::GetSystemWebProxy() - $proxy.Credentials = [System.Net.CredentialCache]::DefaultCredentials - $wc.Proxy = $proxy - return $wc -} - -Write-Host "Preparing to run build script..." - -if(!$PSScriptRoot){ - $PSScriptRoot = Split-Path $MyInvocation.MyCommand.Path -Parent -} - -$TOOLS_DIR = Join-Path $PSScriptRoot "tools" -$ADDINS_DIR = Join-Path $TOOLS_DIR "Addins" -$MODULES_DIR = Join-Path $TOOLS_DIR "Modules" -$NUGET_EXE = Join-Path $TOOLS_DIR "nuget.exe" -$CAKE_EXE = Join-Path $TOOLS_DIR "Cake/Cake.exe" -$NUGET_URL = "https://dist.nuget.org/win-x86-commandline/latest/nuget.exe" -$PACKAGES_CONFIG = Join-Path $TOOLS_DIR "packages.config" -$PACKAGES_CONFIG_MD5 = Join-Path $TOOLS_DIR "packages.config.md5sum" -$ADDINS_PACKAGES_CONFIG = Join-Path $ADDINS_DIR "packages.config" -$MODULES_PACKAGES_CONFIG = Join-Path $MODULES_DIR "packages.config" - -# Make sure tools folder exists -if ((Test-Path $PSScriptRoot) -and !(Test-Path $TOOLS_DIR)) { - Write-Verbose -Message "Creating tools directory..." - New-Item -Path $TOOLS_DIR -Type directory | out-null -} - -# Make sure that packages.config exist. -if (!(Test-Path $PACKAGES_CONFIG)) { - Write-Verbose -Message "Downloading packages.config..." - try { - $wc = GetProxyEnabledWebClient - $wc.DownloadFile("https://cakebuild.net/download/bootstrapper/packages", $PACKAGES_CONFIG) } catch { - Throw "Could not download packages.config." - } -} - -# Try find NuGet.exe in path if not exists -if (!(Test-Path $NUGET_EXE)) { - Write-Verbose -Message "Trying to find nuget.exe in PATH..." - $existingPaths = $Env:Path -Split ';' | Where-Object { (![string]::IsNullOrEmpty($_)) -and (Test-Path $_ -PathType Container) } - $NUGET_EXE_IN_PATH = Get-ChildItem -Path $existingPaths -Filter "nuget.exe" | Select -First 1 - if ($NUGET_EXE_IN_PATH -ne $null -and (Test-Path $NUGET_EXE_IN_PATH.FullName)) { - Write-Verbose -Message "Found in PATH at $($NUGET_EXE_IN_PATH.FullName)." - $NUGET_EXE = $NUGET_EXE_IN_PATH.FullName - } -} - -# Try download NuGet.exe if not exists -if (!(Test-Path $NUGET_EXE)) { - Write-Verbose -Message "Downloading NuGet.exe..." - try { - $wc = GetProxyEnabledWebClient - $wc.DownloadFile($NUGET_URL, $NUGET_EXE) - } catch { - Throw "Could not download NuGet.exe." - } -} - -# Save nuget.exe path to environment to be available to child processed -$ENV:NUGET_EXE = $NUGET_EXE - -# Restore tools from NuGet? -if(-Not $SkipToolPackageRestore.IsPresent) { - Push-Location - Set-Location $TOOLS_DIR - - # Check for changes in packages.config and remove installed tools if true. - [string] $md5Hash = MD5HashFile($PACKAGES_CONFIG) - if((!(Test-Path $PACKAGES_CONFIG_MD5)) -Or - ($md5Hash -ne (Get-Content $PACKAGES_CONFIG_MD5 ))) { - Write-Verbose -Message "Missing or changed package.config hash..." - Remove-Item * -Recurse -Exclude packages.config,nuget.exe - } - - Write-Verbose -Message "Restoring tools from NuGet..." - $NuGetOutput = Invoke-Expression "&`"$NUGET_EXE`" install -ExcludeVersion -OutputDirectory `"$TOOLS_DIR`"" - - if ($LASTEXITCODE -ne 0) { - Throw "An error occurred while restoring NuGet tools." - } - else - { - $md5Hash | Out-File $PACKAGES_CONFIG_MD5 -Encoding "ASCII" - } - Write-Verbose -Message ($NuGetOutput | out-string) - - Pop-Location -} - -# Restore addins from NuGet -if (Test-Path $ADDINS_PACKAGES_CONFIG) { - Push-Location - Set-Location $ADDINS_DIR - - Write-Verbose -Message "Restoring addins from NuGet..." - $NuGetOutput = Invoke-Expression "&`"$NUGET_EXE`" install -ExcludeVersion -OutputDirectory `"$ADDINS_DIR`"" - - if ($LASTEXITCODE -ne 0) { - Throw "An error occurred while restoring NuGet addins." - } - - Write-Verbose -Message ($NuGetOutput | out-string) - - Pop-Location -} - -# Restore modules from NuGet -if (Test-Path $MODULES_PACKAGES_CONFIG) { - Push-Location - Set-Location $MODULES_DIR - - Write-Verbose -Message "Restoring modules from NuGet..." - $NuGetOutput = Invoke-Expression "&`"$NUGET_EXE`" install -ExcludeVersion -OutputDirectory `"$MODULES_DIR`"" - - if ($LASTEXITCODE -ne 0) { - Throw "An error occurred while restoring NuGet modules." - } - - Write-Verbose -Message ($NuGetOutput | out-string) - - Pop-Location -} - -# Make sure that Cake has been installed. -if (!(Test-Path $CAKE_EXE)) { - Throw "Could not find Cake.exe at $CAKE_EXE" -} - - - -# Build Cake arguments -$cakeArguments = @("$Script"); -if ($Target) { $cakeArguments += "-target=$Target" } -if ($Configuration) { $cakeArguments += "-configuration=$Configuration" } -if ($Verbosity) { $cakeArguments += "-verbosity=$Verbosity" } -if ($ShowDescription) { $cakeArguments += "-showdescription" } -if ($DryRun) { $cakeArguments += "-dryrun" } -if ($Experimental) { $cakeArguments += "-experimental" } -if ($Mono) { $cakeArguments += "-mono" } -$cakeArguments += $ScriptArgs - -# Start Cake -Write-Host "Running build script..." -&$CAKE_EXE $cakeArguments -exit $LASTEXITCODE \ No newline at end of file +Write-Host "Running Build" +dotnet cake $SCRIPT_NAME @args +if ($LASTEXITCODE -ne 0) { exit $LASTEXITCODE } \ No newline at end of file diff --git a/build.sh b/build.sh index 4b690aa..ba5d0c9 100644 --- a/build.sh +++ b/build.sh @@ -1,103 +1,11 @@ -#!/usr/bin/env bash -########################################################################## -# This is the Cake bootstrapper script for Linux and OS X. -# This file was downloaded from https://github.com/cake-build/resources -# Feel free to change this file to fit your needs. -########################################################################## +#!/bin/bash +SCRIPT_NAME="recipe.cake" -# Define directories. -SCRIPT_DIR=$( cd "$( dirname "${BASH_SOURCE[0]}" )" && pwd ) -TOOLS_DIR=$SCRIPT_DIR/tools -NUGET_EXE=$TOOLS_DIR/nuget.exe -NUGET_URL=https://dist.nuget.org/win-x86-commandline/latest/nuget.exe -CAKE_VERSION=0.32.1 -CAKE_EXE=$TOOLS_DIR/Cake.$CAKE_VERSION/Cake.exe +echo "Restoring .NET Core tools" +dotnet tool restore -# Temporarily skip verification of addins. -export CAKE_SETTINGS_SKIPVERIFICATION='true' +echo "Bootstrapping Cake" +dotnet cake $SCRIPT_NAME --bootstrap -# Define default arguments. -SCRIPT="recipe.cake" -TARGET="Default" -CONFIGURATION="Release" -VERBOSITY="verbose" -DRYRUN= -SCRIPT_ARGUMENTS=() - -# Parse arguments. -for i in "$@"; do - case $1 in - -t|--target) TARGET="$2"; shift ;; - -c|--configuration) CONFIGURATION="$2"; shift ;; - -v|--verbosity) VERBOSITY="$2"; shift ;; - -d|--dryrun) DRYRUN="-dryrun" ;; - --) shift; SCRIPT_ARGUMENTS+=("$@"); break ;; - *) SCRIPT_ARGUMENTS+=("$1") ;; - esac - shift -done - -# Make sure the tools folder exist. -if [ ! -d "$TOOLS_DIR" ]; then - mkdir "$TOOLS_DIR" -fi - -# Print Mono version. -echo "Mono version:" -mono --version -echo "" - -########################################################################### -# INSTALL .NET CORE CLI -########################################################################### - -echo "Installing .NET CLI..." -if [ ! -d "$SCRIPT_DIR/.dotnet" ]; then - mkdir "$SCRIPT_DIR/.dotnet" -fi -curl -Lsfo "$SCRIPT_DIR/.dotnet/dotnet-install.sh" https://dot.net/v1/dotnet-install.sh -sudo bash "$SCRIPT_DIR/.dotnet/dotnet-install.sh" --version 2.1.400 --install-dir .dotnet --no-path -export PATH="$SCRIPT_DIR/.dotnet":$PATH -export DOTNET_SKIP_FIRST_TIME_EXPERIENCE=1 -export DOTNET_CLI_TELEMETRY_OPTOUT=1 -export DOTNET_SYSTEM_NET_HTTP_USESOCKETSHTTPHANDLER=0 -"$SCRIPT_DIR/.dotnet/dotnet" --info - -########################################################################### -# INSTALL NUGET -########################################################################### - -# Download NuGet if it does not exist. -if [ ! -f "$NUGET_EXE" ]; then - echo "Downloading NuGet..." - curl -Lsfo "$NUGET_EXE" $NUGET_URL - if [ $? -ne 0 ]; then - echo "An error occurred while downloading nuget.exe." - exit 1 - fi -fi - -########################################################################### -# INSTALL CAKE -########################################################################### - -if [ ! -f "$CAKE_EXE" ]; then - mono "$NUGET_EXE" install Cake -Version $CAKE_VERSION -OutputDirectory "$TOOLS_DIR" - if [ $? -ne 0 ]; then - echo "An error occurred while installing Cake." - exit 1 - fi -fi - -# Make sure that Cake has been installed. -if [ ! -f "$CAKE_EXE" ]; then - echo "Could not find Cake.exe at '$CAKE_EXE'." - exit 1 -fi - -########################################################################### -# RUN BUILD SCRIPT -########################################################################### - -# Start Cake -exec mono "$CAKE_EXE" $SCRIPT --verbosity=$VERBOSITY --configuration=$CONFIGURATION --target=$TARGET $DRYRUN "${SCRIPT_ARGUMENTS[@]}" +echo "Running Build" +dotnet cake $SCRIPT_NAME "$@" \ No newline at end of file diff --git a/docs/features.md b/docs/features.md index a766498..05a05bf 100644 --- a/docs/features.md +++ b/docs/features.md @@ -33,5 +33,14 @@ The [Cake.Issues.DupFinder addin] provides the following features: | | `IIssue.Rule` | Always `dupFinder` | | | `IIssue.RuleUrl` | | +## Additional information + +The following values are provided in the [IIssue.AdditionalInformation] property: + +| Key | Description | +|--------|----------------------------------------------------------------------------------| +| `cost` | Cost of the duplicate. The bigger the duplicated code block the higher the cost. | + [JetBrains dupFinder]: https://www.jetbrains.com/help/resharper/dupFinder.html [Cake.Issues.DupFinder addin]: https://www.nuget.org/packages/Cake.Issues.DupFinder +[IIssue.AdditionalInformation]: /api/Cake.Issues/IIssue/FE50CEF3 diff --git a/nuspec/nuget/Cake.Issues.DupFinder.nuspec b/nuspec/nuget/Cake.Issues.DupFinder.nuspec index d07f3a9..77f916d 100644 --- a/nuspec/nuget/Cake.Issues.DupFinder.nuspec +++ b/nuspec/nuget/Cake.Issues.DupFinder.nuspec @@ -24,15 +24,18 @@ See the Project Site for an overview of the whole ecosystem of addins for workin Copyright © janniksam and contributors cake cake-addin cake-issues cake-issueprovider codeanalysis dupFinder - https://github.com/cake-contrib/Cake.Issues.DupFinder/releases/tag/1.0.0 + https://github.com/cake-contrib/Cake.Issues.DupFinder/releases/tag/2.0.0 - - - - - - + + + + + + + + + diff --git a/recipe.cake b/recipe.cake index a99e0d9..9d6b0e4 100644 --- a/recipe.cake +++ b/recipe.cake @@ -1,4 +1,8 @@ -#load nuget:?package=Cake.Recipe&version=1.1.1 +#load nuget:https://pkgs.dev.azure.com/cake-contrib/Home/_packaging/addins/nuget/v3/index.json?package=Cake.Recipe&version=3.0.0-beta0001-0007&prerelease + +//************************************************************************************************* +// Settings +//************************************************************************************************* Environment.SetVariableNames(); @@ -10,21 +14,19 @@ BuildParameters.SetParameters( repositoryOwner: "cake-contrib", repositoryName: "Cake.Issues.DupFinder", appVeyorAccountName: "cakecontrib", - shouldGenerateDocumentation: false, - shouldRunCodecov: false, - shouldRunGitVersion: true); + shouldRunCoveralls: false, // Disabled because it's currently failing + shouldGenerateDocumentation: false); BuildParameters.PrintParameters(Context); ToolSettings.SetToolSettings( context: Context, - dupFinderExcludePattern: new string[] - { - BuildParameters.RootDirectoryPath + "/src/Cake.Issues.DupFinder*/**/*.AssemblyInfo.cs", - BuildParameters.RootDirectoryPath + "/src/Cake.Issues.DupFinder.Tests/**/*.cs" - }, testCoverageFilter: "+[*]* -[xunit.*]* -[Cake.Core]* -[Cake.Testing]* -[*.Tests]* -[Cake.Issues]* -[Cake.Issues.Testing]* -[Shouldly]*", testCoverageExcludeByAttribute: "*.ExcludeFromCodeCoverage*", testCoverageExcludeByFile: "*/*Designer.cs;*/*.g.cs;*/*.g.i.cs"); +//************************************************************************************************* +// Execution +//************************************************************************************************* + Build.RunDotNetCore(); diff --git a/src/Cake.Issues.DupFinder.Tests/Cake.Issues.DupFinder.Tests.csproj b/src/Cake.Issues.DupFinder.Tests/Cake.Issues.DupFinder.Tests.csproj index 91dfba8..12b850e 100644 --- a/src/Cake.Issues.DupFinder.Tests/Cake.Issues.DupFinder.Tests.csproj +++ b/src/Cake.Issues.DupFinder.Tests/Cake.Issues.DupFinder.Tests.csproj @@ -1,6 +1,6 @@ - netcoreapp3.1;net461 + netcoreapp3.1 false Cake.Issues Tests for the Cake.Issues.DupFinder addin @@ -17,17 +17,17 @@ - - - + + + all runtime; build; native; contentfiles; analyzers - - + + all runtime; build; native; contentfiles; analyzers diff --git a/src/Cake.Issues.DupFinder.sln b/src/Cake.Issues.DupFinder.sln index e0fc9ac..7e684ea 100644 --- a/src/Cake.Issues.DupFinder.sln +++ b/src/Cake.Issues.DupFinder.sln @@ -1,11 +1,11 @@  Microsoft Visual Studio Solution File, Format Version 12.00 -# Visual Studio Version 16 -VisualStudioVersion = 16.0.30002.166 +# Visual Studio Version 17 +VisualStudioVersion = 17.2.32526.322 MinimumVisualStudioVersion = 10.0.40219.1 Project("{9A19103F-16F7-4668-BE54-9A1E7A4F7556}") = "Cake.Issues.DupFinder", "Cake.Issues.DupFinder\Cake.Issues.DupFinder.csproj", "{11CE5F92-4901-4AFF-A470-62664ABD986B}" EndProject -Project("{FAE04EC0-301F-11D3-BF4B-00C04F79EFBC}") = "Cake.Issues.DupFinder.Tests", "Cake.Issues.DupFinder.Tests\Cake.Issues.DupFinder.Tests.csproj", "{6AB9A78B-0B02-490B-9100-F331559E8CBD}" +Project("{9A19103F-16F7-4668-BE54-9A1E7A4F7556}") = "Cake.Issues.DupFinder.Tests", "Cake.Issues.DupFinder.Tests\Cake.Issues.DupFinder.Tests.csproj", "{6AB9A78B-0B02-490B-9100-F331559E8CBD}" EndProject Global GlobalSection(SolutionConfigurationPlatforms) = preSolution diff --git a/src/Cake.Issues.DupFinder/Cake.Issues.DupFinder.csproj b/src/Cake.Issues.DupFinder/Cake.Issues.DupFinder.csproj index 1a0e44d..6bc3584 100644 --- a/src/Cake.Issues.DupFinder/Cake.Issues.DupFinder.csproj +++ b/src/Cake.Issues.DupFinder/Cake.Issues.DupFinder.csproj @@ -1,7 +1,7 @@ - netstandard2.0;net461 + netcoreapp3.1;net5.0;net6.0 Cake.Issues JetBrains dupFinder Code support for the Cake.Issues Addin for Cake Build Automation System janniksam @@ -21,9 +21,9 @@ - - - + + + all runtime; build; native; contentfiles; analyzers; buildtransitive diff --git a/src/Cake.Issues.DupFinder/Properties/AssemblyInfo.cs b/src/Cake.Issues.DupFinder/Properties/ProjectInfo.cs similarity index 100% rename from src/Cake.Issues.DupFinder/Properties/AssemblyInfo.cs rename to src/Cake.Issues.DupFinder/Properties/ProjectInfo.cs diff --git a/tools/packages.config b/tools/packages.config deleted file mode 100644 index 997c0e1..0000000 --- a/tools/packages.config +++ /dev/null @@ -1,4 +0,0 @@ - - - -