Bump YamlDotNet from 13.7.1 to 15.1.1 in /src #388
Annotations
1 error and 11 warnings
Validate licenses
Process completed with exit code 1.
|
Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: actions/checkout@v3, actions/setup-dotnet@v3. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.
|
dotnet build:
src/ActionsImporter.ThirdPartyLicenseGenerator/Program.cs#L29
'File.ReadAllLines(string)' synchronously blocks. Await 'File.ReadAllLinesAsync(string, CancellationToken)' instead. (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1849)
|
dotnet build:
src/ActionsImporter.ThirdPartyLicenseGenerator/Program.cs#L218
Type 'LicenseInfo' can be sealed because it has no subtypes in its containing assembly and is not externally visible (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1852)
|
dotnet build:
src/ActionsImporter/Services/DockerService.cs#L74
Avoid creating a new 'JsonSerializerOptions' instance for every serialization operation. Cache and reuse instances instead. (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1869)
|
dotnet build:
src/ActionsImporter/Services/DockerService.cs#L70
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)
|
dotnet build:
src/ActionsImporter/Program.cs#L59
Prefer using 'string.Equals(string, StringComparison)' to perform a case-insensitive comparison, but keep in mind that this might cause subtle changes in behavior, so make sure to conduct thorough testing after applying the suggestion, or if culturally sensitive comparison is not required, consider using 'StringComparison.OrdinalIgnoreCase' (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1862)
|
dotnet build:
src/ActionsImporter/Services/ConfigurationService.cs#L34
Prefer comparing 'Count' to 0 rather than using 'Any()', both for clarity and for performance (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1860)
|
dotnet build:
src/ActionsImporter/Services/ConfigurationService.cs#L42
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)
|
dotnet build:
src/ActionsImporter/Commands/ListFeatures.cs#L16
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)
|
dotnet build:
src/ActionsImporter/Commands/Common.cs#L33
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)
|
dotnet build:
src/ActionsImporter/Commands/Common.cs#L41
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)
|
The logs for this run have expired and are no longer available.
Loading