Merge pull request #215 from manuc66/dependabot/nuget/JsonSubTypes.Te… #59
manuc66Annotations
21 warnings
|
modern
Node.js 20 is deprecated. The following actions target Node.js 20 but are being forced to run on Node.js 24: actions/github-script@60a0d83039c74a4aee543508d2ffcb1c3799cdea. For more information see: https://github.blog/changelog/2025-09-19-deprecation-of-node-20-on-github-actions-runners/
|
|
analysis:
JsonSubTypes.Text.Json/JsonSubtypes.cs#L183
Rename parameter 'objectType' to 'typeToConvert' to match the base class declaration.
|
|
analysis:
JsonSubTypes.Text.Json/NullableDictionary.cs#L35
Use a constructor overloads that allows a more meaningful exception message to be provided.
|
|
analysis:
JsonSubTypes.Text.Json/JsonSubTypesTypeResolution.cs#L27
Refactor 'SearchAssemblies' into a method, properties should not copy collections.
|
|
analysis:
JsonSubTypes.Tests/GenericTests.cs#L41
Add at least one assertion to this test case.
|
|
analysis:
JsonSubTypes/JsonSubtypes.cs#L526
Refactor this method to reduce its Cognitive Complexity from 17 to the 15 allowed.
|
|
analysis:
JsonSubTypes/JsonSubtypes.cs#L333
Make 'GetTypeResolver' a static method.
|
|
analysis:
JsonSubTypes/JsonSubtypesByDiscriminatorValueConverter.cs#L119
Remove this set, which updates a 'static' field from an instance method.
|
|
analysis:
JsonSubTypes/JsonSubtypesByDiscriminatorValueConverter.cs#L118
Remove this set, which updates a 'static' field from an instance method.
|
|
analysis:
JsonSubTypes/JsonSubtypesByDiscriminatorValueConverter.cs#L106
Remove this set, which updates a 'static' field from an instance property.
|
|
analysis:
JsonSubTypes/NullableDictionary.cs#L35
Use a constructor overloads that allows a more meaningful exception message to be provided.
|
|
analysis:
JsonSubTypes/JsonSubtypesByDiscriminatorValueConverter.cs#L119
Remove this set, which updates a 'static' field from an instance method.
|
|
analysis:
JsonSubTypes/JsonSubtypesByDiscriminatorValueConverter.cs#L118
Remove this set, which updates a 'static' field from an instance method.
|
|
analysis:
JsonSubTypes/JsonSubtypesByDiscriminatorValueConverter.cs#L106
Remove this set, which updates a 'static' field from an instance property.
|
|
analysis:
JsonSubTypes/NullableDictionary.cs#L35
Use a constructor overloads that allows a more meaningful exception message to be provided.
|
|
analysis:
JsonSubTypes/JsonSubtypes.cs#L526
Refactor this method to reduce its Cognitive Complexity from 17 to the 15 allowed.
|
|
analysis:
JsonSubTypes/JsonSubtypes.cs#L333
Make 'GetTypeResolver' a static method.
|
|
analysis:
JsonSubTypes/JsonSubtypesByDiscriminatorValueConverter.cs#L119
Remove this set, which updates a 'static' field from an instance method.
|
|
analysis:
JsonSubTypes/JsonSubtypesByDiscriminatorValueConverter.cs#L118
Remove this set, which updates a 'static' field from an instance method.
|
|
analysis:
JsonSubTypes/JsonSubtypesByDiscriminatorValueConverter.cs#L106
Remove this set, which updates a 'static' field from an instance property.
|
|
analysis:
JsonSubTypes/NullableDictionary.cs#L35
Use a constructor overloads that allows a more meaningful exception message to be provided.
|