Antag Selection Logging - #5847
Merged
walksanatora merged 6 commits intoSep 2, 2026
Merged
Annotations
14 warnings
|
Complete job
Node.js 20 is deprecated. The following actions target Node.js 20 but are being forced to run on Node.js 24: space-wizards/submodule-dependency@v0.1.5. For more information see: https://github.blog/changelog/2025-09-19-deprecation-of-node-20-on-github-actions-runners/
|
|
Report C# diagnostics on changed lines:
Content.Server/Antag/AntagSelectionSystem.API.Assignment.cs#L342
[C# diagnostics] reported by reviewdog 🐶
Use expression body for method
Raw Output:
{"level":"warning","locations":[{"physicalLocation":{"artifactLocation":{"uri":"file:///home/runner/work/space-station-14/space-station-14/Content.Server/Antag/AntagSelectionSystem.API.Assignment.cs"},"region":{"endColumn":79,"endLine":342,"startColumn":9,"startLine":342}}}],"message":{"text":"Use expression body for method"},"properties":{},"relatedLocations":[{"physicalLocation":{"artifactLocation":{"uri":"file:///home/runner/work/space-station-14/space-station-14/Content.Server/Antag/AntagSelectionSystem.API.Assignment.cs"},"region":{"endColumn":6,"endLine":343,"startColumn":5,"startLine":337}}}],"ruleId":"IDE0022","ruleIndex":663}
|
|
Report C# diagnostics on changed lines:
Content.Server/Antag/AntagSelectionSystem.Starlight.cs#L49
[C# diagnostics] reported by reviewdog 🐶
Avoid multiple blank lines
Raw Output:
{"level":"warning","locations":[{"physicalLocation":{"artifactLocation":{"uri":"file:///home/runner/work/space-station-14/space-station-14/Content.Server/Antag/AntagSelectionSystem.Starlight.cs"},"region":{"endColumn":1,"endLine":49,"startColumn":1,"startLine":49}}}],"message":{"text":"Avoid multiple blank lines"},"properties":{},"relatedLocations":[{"physicalLocation":{"artifactLocation":{"uri":"file:///home/runner/work/space-station-14/space-station-14/Content.Server/Antag/AntagSelectionSystem.Starlight.cs"},"region":{"endColumn":14,"endLine":52,"startColumn":5,"startLine":52}}}],"ruleId":"IDE2000","ruleIndex":655}
|
|
Report C# diagnostics on changed lines:
Content.Server/Antag/AntagSelectionSystem.Starlight.cs#L486
[C# diagnostics] reported by reviewdog 🐶
Consecutive braces must not have blank line between them
Raw Output:
{"level":"warning","locations":[{"physicalLocation":{"artifactLocation":{"uri":"file:///home/runner/work/space-station-14/space-station-14/Content.Server/Antag/AntagSelectionSystem.Starlight.cs"},"region":{"endColumn":2,"endLine":486,"startColumn":1,"startLine":486}}}],"message":{"text":"Consecutive braces must not have blank line between them"},"properties":{},"ruleId":"IDE2002","ruleIndex":656}
|
|
Build Project and collect C# diagnostics:
Content.Server.Database/SnakeCaseNaming.cs#L13
Convert to file-scoped namespace (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/style-rules/ide0161)
|
|
Build Project and collect C# diagnostics:
Content.Server.Database/ModelSqlite.cs#L14
Convert to file-scoped namespace (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/style-rules/ide0161)
|
|
Build Project and collect C# diagnostics:
Content.Server.Database/ModelPostgres.cs#L9
Convert to file-scoped namespace (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/style-rules/ide0161)
|
|
Build Project and collect C# diagnostics:
Content.Server.Database/Model.cs#L14
Convert to file-scoped namespace (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/style-rules/ide0161)
|
|
Build Project and collect C# diagnostics:
Content.Server.Database/DataModelBase.cs#L13
Consecutive braces must not have blank line between them (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/style-rules/ide2002)
|
|
Build Project and collect C# diagnostics:
Content.Server.Database/DataModelBase.cs#L36
Consecutive braces must not have blank line between them (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/style-rules/ide2002)
|
|
Build Project and collect C# diagnostics:
Content.Shared.Database/TypedHwid.cs#L46
Use expression body for method (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/style-rules/ide0022)
|
|
Build Project and collect C# diagnostics:
Content.Shared.Database/TypedHwid.cs#L46
Parentheses should be added for clarity (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/style-rules/ide0048)
|
|
Build Project and collect C# diagnostics:
Content.Shared.Database/NoteSeverity.cs#L1
Convert to file-scoped namespace (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/style-rules/ide0161)
|
|
Build Project and collect C# diagnostics:
Content.PatreonParser/Program.cs#L83
Use pattern matching (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/style-rules/ide0078)
|
background
wait
wait-all
cancel
parallel
Loading