Skip to content

CodeQL

CodeQL #316

Triggered via schedule October 6, 2024 22:49
Status Success
Total duration 2m 45s
Artifacts

codeql-analysis.yml

on: schedule
Fit to window
Zoom out
Zoom in

Annotations

12 warnings
Analyze
The following actions use a deprecated Node.js version and will be forced to run on node20: actions/checkout@v3, github/codeql-action/init@v2, github/codeql-action/autobuild@v2, github/codeql-action/analyze@v2. For more info: https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/
Analyze
CodeQL Action v2 will be deprecated on December 5th, 2024. Please update all occurrences of the CodeQL Action in your workflow files to v3. For more information, see https://github.blog/changelog/2024-01-12-code-scanning-deprecation-of-codeql-action-v2/
Analyze: src/PetroGlyph.Games.EawFoc/test/PlayableObjectTest.cs#L56
Non-nullable property 'Game' must contain a non-null value when exiting constructor. Consider adding the 'required' modifier or declaring the property as nullable.
Analyze: src/PetroGlyph.Games.EawFoc/test/GameServices/SteamWorkshopWebpageDownloaderTest.cs#L10
Support for 'async void' unit tests is being removed from xUnit.net v3. To simplify upgrading, convert the test to 'async Task' instead. (https://xunit.net/xunit.analyzers/rules/xUnit1048)
Analyze: src/PetroGlyph.Games.EawFoc/test/FileBasedLanguageFinderTest.cs#L40
Comparing an instance of System.Collections.Generic.List<EawModinfo.Spec.ILanguageInfo> with an instance of System.Collections.Generic.ISet<EawModinfo.Spec.ILanguageInfo> has undefined results, because the order of items in the set is not predictable. Create a stable order for the set (i.e., by using OrderBy from Linq). (https://xunit.net/xunit.analyzers/rules/xUnit2027)
Analyze: src/PetroGlyph.Games.EawFoc/test/GameServices/GameLanguageFinderTest.cs#L63
Do not use Assert.Equal() to check for collection size. Use Assert.Single instead. (https://xunit.net/xunit.analyzers/rules/xUnit2013)
Analyze: sample/SampleApplication/Program.cs#L34
Converting null literal or possible null value to non-nullable type.
Analyze: src/PetroGlyph.Games.EawFoc/test/PlayableObjectTest.cs#L56
Non-nullable property 'Game' must contain a non-null value when exiting constructor. Consider adding the 'required' modifier or declaring the property as nullable.
Analyze: src/PetroGlyph.Games.EawFoc/test/GameServices/SteamWorkshopWebpageDownloaderTest.cs#L10
Support for 'async void' unit tests is being removed from xUnit.net v3. To simplify upgrading, convert the test to 'async Task' instead. (https://xunit.net/xunit.analyzers/rules/xUnit1048)
Analyze: src/PetroGlyph.Games.EawFoc/test/FileBasedLanguageFinderTest.cs#L40
Comparing an instance of System.Collections.Generic.List<EawModinfo.Spec.ILanguageInfo> with an instance of System.Collections.Generic.ISet<EawModinfo.Spec.ILanguageInfo> has undefined results, because the order of items in the set is not predictable. Create a stable order for the set (i.e., by using OrderBy from Linq). (https://xunit.net/xunit.analyzers/rules/xUnit2027)