Skip to content

Commit fa2ef82

Browse files
authored
Update SDK version to 9.0.100 (#76)
- **New Features** - Updated SDK version to improve compatibility and performance. - Introduced new code analysis rules to enhance code quality and maintainability. - **Bug Fixes** - Disabled the SquiggleCop analyzer due to a known bug, with plans to re-enable it once resolved. - **Documentation** - Minor corrections to rule titles for clarity and consistency. - **Chores** - Enhanced GitHub Actions workflow configuration for better maintainability and configurability. - Updated package version for improved functionality. --------- Signed-off-by: Richard Murillo <[email protected]>
1 parent 3e407ec commit fa2ef82

File tree

12 files changed

+22
-7972
lines changed

12 files changed

+22
-7972
lines changed

Diff for: .github/workflows/main.yml

+4-4
Original file line numberDiff line numberDiff line change
@@ -13,11 +13,11 @@ on:
1313
- main
1414
- 'releases/**'
1515
- 'release/**'
16-
- 'feature/**'
16+
- 'feature/**'
1717
merge_group:
1818
branches:
1919
- main
20-
20+
2121
workflow_call: # Allow to be called from the release workflow
2222
schedule:
2323
- cron: '31 15 * * 0' # Run periodically to keep CodeQL database updated
@@ -134,7 +134,7 @@ jobs:
134134
run: cat artifacts/TestResults/coverage/SummaryGithub.md >> $GITHUB_STEP_SUMMARY
135135
shell: bash
136136

137-
- name: Upload coverage data to Codacy
137+
- name: Upload coverage data to Codacy
138138
if: ${{ runner.os == 'Linux' && env.IS_COVERAGE_ALLOWED == 'true' }}
139139
uses: codacy/[email protected]
140140
with:
@@ -152,7 +152,7 @@ jobs:
152152
- name: Validate performance
153153
shell: pwsh
154154
# Uses ETL which needs Windows
155-
if: ${{ runner.os == 'Windows' }}
155+
if: ${{ runner.os == 'Windows' }}
156156
run: ${{ github.workspace }}/build/scripts/perf/PerfCore.ps1 -v diag -diff -ci
157157

158158
- name: Perform CodeQL Analysis

Diff for: build/targets/codeanalysis/CodeAnalysis.props

-4
Original file line numberDiff line numberDiff line change
@@ -32,10 +32,6 @@
3232
<PrivateAssets>all</PrivateAssets>
3333
<IncludeAssets>runtime; build; native; contentfiles; analyzers; buildtransitive</IncludeAssets>
3434
</PackageReference>
35-
<PackageReference Include="SquiggleCop.Tasks">
36-
<PrivateAssets>all</PrivateAssets>
37-
<IncludeAssets>runtime; build; native; contentfiles; analyzers; buildtransitive</IncludeAssets>
38-
</PackageReference>
3935
<PackageReference Include="Microsoft.VisualStudio.Threading.Analyzers">
4036
<PrivateAssets>all</PrivateAssets>
4137
<IncludeAssets>runtime; build; native; contentfiles; analyzers; buildtransitive</IncludeAssets>

Diff for: build/targets/codeanalysis/CodeAnalysis.targets

-1
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,6 @@
33
<PedanticMode Condition=" '$(PedanticMode)' == '' ">$([MSBuild]::ValueOrDefault('$(ContinuousIntegrationBuild)', 'false'))</PedanticMode>
44
<TreatWarningsAsErrors>$(PedanticMode)</TreatWarningsAsErrors>
55
<MSBuildTreatWarningsAsErrors>$(PedanticMode)</MSBuildTreatWarningsAsErrors>
6-
<SquiggleCop_Enabled>$(PedanticMode)</SquiggleCop_Enabled>
76
</PropertyGroup>
87

98
<Target Name="SetErrorLog" BeforeTargets="CoreCompile">

Diff for: build/targets/codeanalysis/Packages.props

-1
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,6 @@
55
<PackageVersion Include="Roslynator.Analyzers" Version="4.12.4" />
66
<PackageVersion Include="StyleCop.Analyzers" Version="1.2.0-beta.556" />
77
<PackageVersion Include="SonarAnalyzer.CSharp" Version="9.30.0.95878" />
8-
<PackageVersion Include="SquiggleCop.Tasks" Version="1.0.8" />
98
<PackageVersion Include="Microsoft.VisualStudio.Threading.Analyzers" Version="17.10.48" />
109
<PackageVersion Include="ExhaustiveMatching.Analyzer" Version="0.5.0" />
1110
</ItemGroup>

Diff for: global.json

+1-1
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
{
22
"sdk": {
3-
"version": "8.0.403",
3+
"version": "9.0.100",
44
"rollForward": "disable" /* setup-dotnet is broken and we don't want to have differences in CI vs local, so make this disabled */
55
},
66
"msbuild-sdks": {

Diff for: src/EffectiveCSharp.Analyzers/SquiggleCop.Baseline.yaml

-1,561
This file was deleted.

Diff for: src/tools/Dogfood/SquiggleCop.Baseline.yaml

-1,572
This file was deleted.

Diff for: src/tools/PerfDiff/SquiggleCop.Baseline.yaml

-1,575
This file was deleted.
Original file line numberDiff line numberDiff line change
@@ -1,17 +1,16 @@
11
<Project Sdk="Microsoft.NET.Sdk">
2-
3-
<PropertyGroup>
4-
<OutputType>Exe</OutputType>
5-
<TargetFramework>net8.0</TargetFramework>
6-
<IsPackable>false</IsPackable>
7-
</PropertyGroup>
8-
9-
<ItemGroup>
10-
<ProjectReference Include="$(RepoRoot)/src/EffectiveCSharp.Analyzers/EffectiveCSharp.Analyzers.csproj" />
11-
<PackageReference Include="BenchmarkDotNet" />
12-
<PackageReference Include="BenchmarkDotNet.Diagnostics.dotTrace" />
13-
<PackageReference Include="Microsoft.CodeAnalysis.CSharp.Workspaces" />
14-
<PackageReference Include="Microsoft.CodeAnalysis.CSharp.CodeFix.Testing" />
15-
</ItemGroup>
16-
2+
<PropertyGroup>
3+
<OutputType>Exe</OutputType>
4+
<TargetFramework>net9.0</TargetFramework>
5+
<IsPackable>false</IsPackable>
6+
<!-- There are transitive dependencies that need to be updated -->
7+
<NoWarn>$(NoWarn);NU1903</NoWarn>
8+
</PropertyGroup>
9+
<ItemGroup>
10+
<ProjectReference Include="$(RepoRoot)/src/EffectiveCSharp.Analyzers/EffectiveCSharp.Analyzers.csproj" />
11+
<PackageReference Include="BenchmarkDotNet" />
12+
<PackageReference Include="BenchmarkDotNet.Diagnostics.dotTrace" />
13+
<PackageReference Include="Microsoft.CodeAnalysis.CSharp.Workspaces" />
14+
<PackageReference Include="Microsoft.CodeAnalysis.CSharp.CodeFix.Testing" />
15+
</ItemGroup>
1716
</Project>

Diff for: tests/EffectiveCSharp.Analyzers.Benchmarks/SquiggleCop.Baseline.yaml

-1,575
This file was deleted.
Original file line numberDiff line numberDiff line change
@@ -1,20 +1,18 @@
11
<Project Sdk="Microsoft.NET.Sdk">
2-
32
<PropertyGroup>
4-
<TargetFramework>net8.0</TargetFramework>
3+
<TargetFramework>net9.0</TargetFramework>
54
<IsPackable>false</IsPackable>
65
<IsTestProject>true</IsTestProject>
76
<!-- The CompositeAnalyzer triggers RS1036 -->
87
<EnforceExtendedAnalyzerRules>true</EnforceExtendedAnalyzerRules>
8+
<!-- There are transitive dependencies that need to be updated -->
9+
<NoWarn>$(NoWarn);NU1903</NoWarn>
910
</PropertyGroup>
10-
1111
<PropertyGroup Condition="'$(Configuration)|$(Platform)'=='Debug|AnyCPU'">
1212
</PropertyGroup>
13-
1413
<PropertyGroup Condition="'$(Configuration)|$(Platform)'=='Release|AnyCPU'">
1514
<Optimize>false</Optimize>
1615
</PropertyGroup>
17-
1816
<ItemGroup>
1917
<PackageReference Include="GetPackFromProject">
2018
<IncludeAssets>runtime; build; native; contentfiles; analyzers; buildtransitive</IncludeAssets>
@@ -27,5 +25,4 @@
2725
<ItemGroup>
2826
<ProjectReference Include="$(RepoRoot)/src/EffectiveCSharp.Analyzers/EffectiveCSharp.Analyzers.csproj" AddPackageAsOutput="true" />
2927
</ItemGroup>
30-
3128
</Project>

0 commit comments

Comments
 (0)