Merge pull request #38 from AElfProject/release/1.3.0 #11
jason-aelfAnnotations
11 warnings
|
publish
The following actions use a deprecated Node.js version and will be forced to run on node20: actions/checkout@v3, actions/setup-dotnet@v3. For more info: https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/
|
|
publish:
src/AElf.EntityMapping/Linq/EntityMappingCachedReflectionInfo.cs#L7
The annotation for nullable reference types should only be used in code within a '#nullable' annotations context.
|
|
publish:
src/AElf.EntityMapping.Elasticsearch/Linq/GeneratorExpressionTreeVisitor.cs#L371
Type parameter 'T' has the same name as the type parameter from outer type 'GeneratorExpressionTreeVisitor<T>'
|
|
publish:
src/AElf.EntityMapping.Elasticsearch/ElasticsearchCollectionNameProvider.cs#L69
This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread.
|
|
publish:
src/AElf.EntityMapping.Elasticsearch/Services/ElasticIndexService.cs#L100
'MappingsDescriptor.Map(Func<TypeMappingDescriptor<object>, ITypeMapping>)' is obsolete: 'MappingsDescriptor is obsolete please call Map() on the parent descriptor'
|
|
publish:
src/AElf.EntityMapping.Elasticsearch/Services/ElasticIndexService.cs#L98
'PutIndexTemplateDescriptor.Mappings(Func<MappingsDescriptor, ITypeMapping>)' is obsolete: 'Mappings is no longer a dictionary in 7.x, please use the simplified Map() method on this descriptor instead'
|
|
publish:
src/AElf.EntityMapping.Elasticsearch/Linq/NodeVisitor.cs#L257
The variable 'path' is assigned but its value is never used
|
|
publish:
test/AElf.EntityMapping.Elasticsearch.Tests/Sharding/ShardingKeyProviderTest.cs#L138
Because this call is not awaited, execution of the current method continues before the call is completed. Consider applying the 'await' operator to the result of the call.
|
|
publish:
test/AElf.EntityMapping.Elasticsearch.Tests/Repositories/ElasticsearchRepositoryTests.cs#L574
Because this call is not awaited, execution of the current method continues before the call is completed. Consider applying the 'await' operator to the result of the call.
|
|
publish:
test/AElf.EntityMapping.Elasticsearch.Tests/Repositories/ElasticsearchRepositoryTests.cs#L653
Because this call is not awaited, execution of the current method continues before the call is completed. Consider applying the 'await' operator to the result of the call.
|
|
publish:
test/AElf.EntityMapping.Elasticsearch.Tests/Sharding/ShardingKeyProviderTest.cs#L121
Do not use Assert.True() to check for substrings. Use Assert.StartsWith instead. (https://xunit.net/xunit.analyzers/rules/xUnit2009)
|