Skip to content

Update actions/checkout action to v4.1.7 #209

Update actions/checkout action to v4.1.7

Update actions/checkout action to v4.1.7 #209

Triggered via pull request June 12, 2024 19:45
Status Success
Total duration 7m 35s
Artifacts

codeql-analysis.yml

on: pull_request
Matrix: Analyze
Fit to window
Zoom out
Zoom in

Annotations

10 warnings
Analyze (csharp): src/DummyFileCreator/ByteSizeExtensions.cs#L61
Use 'ArgumentOutOfRangeException.ThrowIfNegative' instead of explicitly throwing a new exception instance (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1512)
Analyze (csharp): src/DummyFileCreator/DummyFile.cs#L95
Use recommended dispose pattern to ensure that object created by 'new DummyFileWriter(filepathToCreate, bufferSize)' is disposed on all paths. If possible, wrap the creation within a 'using' statement or a 'using' declaration. Otherwise, use a try-finally pattern, with a dedicated local variable declared before the try region and an unconditional Dispose invocation on non-null value in the 'finally' region, say 'x?.Dispose()'. If the object is explicitly disposed within the try region or the dispose ownership is transfered to another object or method, assign 'null' to the local variable just after such an operation to prevent double dispose in 'finally'. (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca2000)
Analyze (csharp): src/DummyFileCreator.UnitTests/DummyFileUnitTest.cs#L53
Test methods should not call ConfigureAwait(false), as it may bypass parallelization limits. Omit ConfigureAwait, or use ConfigureAwait(true) to avoid CA2007. (https://xunit.net/xunit.analyzers/rules/xUnit1030)
Analyze (csharp): src/DummyFileCreator.UnitTests/DummyFileUnitTest.cs#L55
Test methods should not call ConfigureAwait(false), as it may bypass parallelization limits. Omit ConfigureAwait, or use ConfigureAwait(true) to avoid CA2007. (https://xunit.net/xunit.analyzers/rules/xUnit1030)
Analyze (csharp): src/DummyFileCreator.UnitTests/DummyFileUnitTest.cs#L53
Test methods should not call ConfigureAwait(false), as it may bypass parallelization limits. Omit ConfigureAwait, or use ConfigureAwait(true) to avoid CA2007. (https://xunit.net/xunit.analyzers/rules/xUnit1030)
Analyze (csharp): src/DummyFileCreator.UnitTests/DummyFileUnitTest.cs#L55
Test methods should not call ConfigureAwait(false), as it may bypass parallelization limits. Omit ConfigureAwait, or use ConfigureAwait(true) to avoid CA2007. (https://xunit.net/xunit.analyzers/rules/xUnit1030)
Analyze (csharp): src/DummyFileCreator.UnitTests/DummyFileUnitTest.cs#L38
Test methods should not call ConfigureAwait(false), as it may bypass parallelization limits. Omit ConfigureAwait, or use ConfigureAwait(true) to avoid CA2007. (https://xunit.net/xunit.analyzers/rules/xUnit1030)
Analyze (csharp): src/DummyFileCreator.UnitTests/DummyFileUnitTest.cs#L38
Test methods should not call ConfigureAwait(false), as it may bypass parallelization limits. Omit ConfigureAwait, or use ConfigureAwait(true) to avoid CA2007. (https://xunit.net/xunit.analyzers/rules/xUnit1030)
Analyze (csharp): src/DummyFileCreator.App/Progress/ProgressForm.cs#L115
Dereference of a possibly null reference.
Analyze (csharp): src/DummyFileCreator.Console/Program.cs#L13
Prefer 'static readonly' fields over constant array arguments if the called method is called repeatedly and is not mutating the passed array (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1861)