Remove V2 from the header #365
Annotations
23 warnings
The following actions uses node12 which is deprecated and will be forced to run on node16: actions/checkout@v2. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
|
The following actions use a deprecated Node.js version and will be forced to run on node20: actions/checkout@v2, actions/setup-dotnet@v3, codecov/codecov-action@v3, actions/upload-artifact@v3. For more info: https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/
|
src/LeanCode.Contracts/Binary.cs#L30
Provide a method named 'ToByteArray' or 'FromNullable' as an alternate for operator op_Implicit (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca2225)
|
src/LeanCode.Contracts/Binary.cs#L8
Properties should not return arrays (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1819)
|
src/LeanCode.Contracts/NotificationTagGenerator.cs#L32
'string.IndexOf(char)' has a method overload that takes a 'StringComparison' parameter. Replace this call in 'LeanCode.Contracts.NotificationTagGenerator.GetSimpleName(System.Type)' with a call to 'string.IndexOf(char, System.StringComparison)' for clarity of intent. (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1307)
|
src/LeanCode.Contracts/NotificationTagGenerator.cs#L46
Prefer comparing 'Length' to 0 rather than using 'Any()', both for clarity and for performance (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1860)
|
src/LeanCode.Contracts/Converters/BinaryJsonConverter.cs#L6
Type 'BinaryJsonConverter' 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)
|
src/LeanCode.ContractsGenerator/Analyzers/KnownTypeCheck.cs#L5
Change type of field 'ValidKnownTypeValues' from 'System.Collections.Generic.IReadOnlySet<LeanCode.ContractsGenerator.KnownType>' to 'System.Collections.Generic.HashSet<LeanCode.ContractsGenerator.KnownType>' for improved performance (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1859)
|
src/LeanCode.ContractsGenerator/Compilation/CompilationFailedException.cs#L28
The behavior of 'StringBuilder.AppendLine(ref StringBuilder.AppendInterpolatedStringHandler)' could vary based on the current user's locale settings. Replace this call in 'CompilationFailedException.GetDiagnosticsMessage(ImmutableArray<Diagnostic>)' with a call to 'StringBuilder.AppendLine(IFormatProvider, ref StringBuilder.AppendInterpolatedStringHandler)'. (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1305)
|
src/LeanCode.ContractsGenerator/Compilation/CompilationFailedException.cs#L28
The behavior of 'Diagnostic.GetMessage([IFormatProvider])' could vary based on the current user's locale settings. Provide a value for the 'IFormatProvider' argument. (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1305)
|
src/LeanCode.ContractsGenerator/Generation/NotificationTagGenerator.cs#L33
Possible multiple enumerations of 'IEnumerable' collection. Consider using an implementation that avoids multiple enumerations. (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1851)
|
src/LeanCode.ContractsGenerator/Generation/NotificationTagGenerator.cs#L29
Possible multiple enumerations of 'IEnumerable' collection. Consider using an implementation that avoids multiple enumerations. (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1851)
|
src/LeanCode.Contracts/Binary.cs#L30
Provide a method named 'ToByteArray' or 'FromNullable' as an alternate for operator op_Implicit (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca2225)
|
src/LeanCode.Contracts/Binary.cs#L8
Properties should not return arrays (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1819)
|
src/LeanCode.Contracts/NotificationTagGenerator.cs#L32
'string.IndexOf(char)' has a method overload that takes a 'StringComparison' parameter. Replace this call in 'LeanCode.Contracts.NotificationTagGenerator.GetSimpleName(System.Type)' with a call to 'string.IndexOf(char, System.StringComparison)' for clarity of intent. (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1307)
|
src/LeanCode.Contracts/NotificationTagGenerator.cs#L46
Prefer comparing 'Length' to 0 rather than using 'Any()', both for clarity and for performance (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1860)
|
src/LeanCode.Contracts/Converters/BinaryJsonConverter.cs#L6
Type 'BinaryJsonConverter' 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)
|
src/LeanCode.Contracts/Binary.cs#L30
Provide a method named 'ToByteArray' or 'FromNullable' as an alternate for operator op_Implicit (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca2225)
|
src/LeanCode.Contracts/Binary.cs#L8
Properties should not return arrays (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1819)
|
src/LeanCode.Contracts/Security/AuthorizeWhenHasAnyOfAttribute.cs#L10
Add a public read-only property accessor for positional argument permissions of Attribute AuthorizeWhenHasAnyOfAttribute (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1019)
|
src/LeanCode.Contracts/CommandResult.cs#L20
In externally visible method 'CommandResult CommandResult.NotValid(ValidationResult validationResult)', validate parameter 'validationResult' is non-null before using it. If appropriate, throw an 'ArgumentNullException' when the argument is 'null'. (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1062)
|
src/LeanCode.Contracts/NotificationEnvelope.cs#L16
In externally visible method 'NotificationEnvelope.NotificationEnvelope(Guid id, ITopic topic, object notification)', validate parameter 'topic' is non-null before using it. If appropriate, throw an 'ArgumentNullException' when the argument is 'null'. (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1062)
|
The following artifacts were uploaded using a version of actions/upload-artifact that is scheduled for deprecation: "test_results_6.0.zip", "test_results_7.0.zip".
Please update your workflow to use v4 of the artifact actions.
Learn more: https://github.blog/changelog/2024-04-16-deprecation-notice-v3-of-the-artifact-actions/
|
This job succeeded
Loading