Skip to content
This repository was archived by the owner on Apr 9, 2024. It is now read-only.

Commit b589c90

Browse files
authored
(GH-128) Build against Cake 2.0 (#139)
1 parent abbe2b5 commit b589c90

File tree

4 files changed

+16
-29
lines changed

4 files changed

+16
-29
lines changed

src/Cake.Issues.Reporting.Sarif.Tests/Cake.Issues.Reporting.Sarif.Tests.csproj

Lines changed: 5 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -22,27 +22,17 @@
2222
<Service Include="{82A7F48D-3B50-4B1E-B82E-3ADA8210C358}" />
2323
</ItemGroup>
2424
<ItemGroup>
25-
<PackageReference Include="Cake.Issues">
26-
<Version>1.0.0</Version>
27-
</PackageReference>
28-
<PackageReference Include="Cake.Issues.Testing">
29-
<Version>1.0.0</Version>
30-
</PackageReference>
31-
<PackageReference Include="Cake.Testing">
32-
<Version>1.0.0</Version>
33-
</PackageReference>
34-
<PackageReference Include="Shouldly">
35-
<Version>4.0.3</Version>
36-
</PackageReference>
25+
<PackageReference Include="Cake.Issues" Version="2.0.0-beta0001" />
26+
<PackageReference Include="Cake.Issues.Testing" Version="2.0.0-beta0001" />
27+
<PackageReference Include="Cake.Testing" Version="2.0.0" />
28+
<PackageReference Include="Shouldly" Version="4.0.3" />
3729
<PackageReference Include="StyleCop.Analyzers">
3830
<Version>1.1.118</Version>
3931
<IncludeAssets>runtime; build; native; contentfiles; analyzers</IncludeAssets>
4032
<PrivateAssets>all</PrivateAssets>
4133
</PackageReference>
4234
<PackageReference Include="Microsoft.NET.Test.Sdk" Version="16.10.0" />
43-
<PackageReference Include="xunit">
44-
<Version>2.4.1</Version>
45-
</PackageReference>
35+
<PackageReference Include="xunit" Version="2.4.1" />
4636
<PackageReference Include="xunit.runner.visualstudio">
4737
<Version>2.4.3</Version>
4838
<IncludeAssets>runtime; build; native; contentfiles; analyzers</IncludeAssets>

src/Cake.Issues.Reporting.Sarif/Cake.Issues.Reporting.Sarif.csproj

Lines changed: 3 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -28,13 +28,9 @@
2828
</PropertyGroup>
2929

3030
<ItemGroup>
31-
<PackageReference Include="Cake.Core">
32-
<Version>1.0.0</Version>
33-
</PackageReference>
34-
<PackageReference Include="Cake.Issues">
35-
<Version>1.0.0</Version>
36-
</PackageReference>
37-
<PackageReference Include="Cake.Issues.Reporting" Version="1.0.0" />
31+
<PackageReference Include="Cake.Core" Version="2.0.0" />
32+
<PackageReference Include="Cake.Issues" Version="2.0.0-beta0001" />
33+
<PackageReference Include="Cake.Issues.Reporting" Version="2.0.0-beta0001" />
3834
<PackageReference Include="Microsoft.CodeAnalysis.NetAnalyzers" Version="5.0.3">
3935
<PrivateAssets>all</PrivateAssets>
4036
<IncludeAssets>runtime; build; native; contentfiles; analyzers; buildtransitive</IncludeAssets>

src/Cake.Issues.Reporting.Sarif/SarifIssueReportGenerator.cs

Lines changed: 7 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -117,7 +117,7 @@ private Result GetResult(IIssue issue)
117117
var result =
118118
new Result
119119
{
120-
RuleId = issue.Rule,
120+
RuleId = issue.RuleId,
121121
Message =
122122
new Message
123123
{
@@ -135,20 +135,21 @@ private Result GetResult(IIssue issue)
135135

136136
if (issue.RuleUrl != null)
137137
{
138-
if (!string.IsNullOrEmpty(issue.Rule))
138+
if (!string.IsNullOrEmpty(issue.RuleId))
139139
{
140-
if (!this.ruleIndices.ContainsKey(issue.Rule))
140+
if (!this.ruleIndices.ContainsKey(issue.RuleId))
141141
{
142-
this.ruleIndices.Add(issue.Rule, this.rules.Count);
142+
this.ruleIndices.Add(issue.RuleId, this.rules.Count);
143143
this.rules.Add(
144144
new ReportingDescriptor
145145
{
146-
Id = issue.Rule,
146+
Id = issue.RuleId,
147+
Name = issue.RuleName,
147148
HelpUri = issue.RuleUrl,
148149
});
149150
}
150151

151-
result.RuleIndex = this.ruleIndices[issue.Rule];
152+
result.RuleIndex = this.ruleIndices[issue.RuleId];
152153
}
153154
else
154155
{

tests/.config/dotnet-tools.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@
33
"isRoot": true,
44
"tools": {
55
"cake.tool": {
6-
"version": "1.0.0",
6+
"version": "2.0.0",
77
"commands": [
88
"dotnet-cake"
99
]

0 commit comments

Comments
 (0)