Smarter cleaning rules for packages with SemVer #275
Triggered via pull request
September 23, 2024 18:48
ErikApption
Status
Success
Total duration
3m 16s
Artifacts
–
Annotations
20 warnings
|
build (ubuntu-latest):
src/BaGetter.Core/Indexing/IPackageDeletionService.cs#L13
XML comment on 'IPackageDeletionService.DeleteOldVersionsAsync(Package, uint?, uint?, uint?, uint?, CancellationToken)' has a paramref tag for 'maxPath', but there is no parameter by that name
|
|
build (ubuntu-latest):
src/BaGetter.Core/Indexing/PackageDeletionService.cs#L94
Member 'GetValidVersions' does not access instance data and can be marked as static (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1822)
|
|
build (ubuntu-latest):
src/BaGetter.Tencent/TencentStorageOptions.cs#L8
Non-nullable property 'AppId' must contain a non-null value when exiting constructor. Consider adding the 'required' modifier or declaring the property as nullable.
|
|
build (ubuntu-latest):
src/BaGetter.Tencent/TencentStorageOptions.cs#L10
Non-nullable property 'SecretId' must contain a non-null value when exiting constructor. Consider adding the 'required' modifier or declaring the property as nullable.
|
|
build (ubuntu-latest):
src/BaGetter.Tencent/TencentStorageOptions.cs#L12
Non-nullable property 'SecretKey' must contain a non-null value when exiting constructor. Consider adding the 'required' modifier or declaring the property as nullable.
|
|
build (ubuntu-latest):
src/BaGetter.Tencent/TencentStorageOptions.cs#L14
Non-nullable property 'Region' must contain a non-null value when exiting constructor. Consider adding the 'required' modifier or declaring the property as nullable.
|
|
build (ubuntu-latest):
src/BaGetter.Tencent/TencentStorageOptions.cs#L16
Non-nullable property 'BucketName' must contain a non-null value when exiting constructor. Consider adding the 'required' modifier or declaring the property as nullable.
|
|
build (ubuntu-latest):
src/BaGetter.Tencent/TencentApplicationExtensions.cs#L27
Possible null reference return.
|
|
build (ubuntu-latest):
src/BaGetter.Tencent/TencentStorageService.cs#L18
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.
|
|
build (ubuntu-latest):
src/BaGetter.Tencent/TencentStorageService.cs#L29
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.
|
|
build (windows-latest):
src/BaGetter.Core/Indexing/IPackageDeletionService.cs#L13
XML comment on 'IPackageDeletionService.DeleteOldVersionsAsync(Package, uint?, uint?, uint?, uint?, CancellationToken)' has a paramref tag for 'maxPath', but there is no parameter by that name
|
|
build (windows-latest):
src/BaGetter.Core/Indexing/PackageDeletionService.cs#L94
Member 'GetValidVersions' does not access instance data and can be marked as static (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1822)
|
|
build (windows-latest):
src/BaGetter.Tencent/TencentApplicationExtensions.cs#L27
Possible null reference return.
|
|
build (windows-latest):
src/BaGetter.Tencent/TencentStorageService.cs#L18
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.
|
|
build (windows-latest):
src/BaGetter.Tencent/TencentStorageOptions.cs#L8
Non-nullable property 'AppId' must contain a non-null value when exiting constructor. Consider adding the 'required' modifier or declaring the property as nullable.
|
|
build (windows-latest):
src/BaGetter.Tencent/TencentStorageOptions.cs#L10
Non-nullable property 'SecretId' must contain a non-null value when exiting constructor. Consider adding the 'required' modifier or declaring the property as nullable.
|
|
build (windows-latest):
src/BaGetter.Tencent/TencentStorageOptions.cs#L12
Non-nullable property 'SecretKey' must contain a non-null value when exiting constructor. Consider adding the 'required' modifier or declaring the property as nullable.
|
|
build (windows-latest):
src/BaGetter.Tencent/TencentStorageOptions.cs#L14
Non-nullable property 'Region' must contain a non-null value when exiting constructor. Consider adding the 'required' modifier or declaring the property as nullable.
|
|
build (windows-latest):
src/BaGetter.Tencent/TencentStorageOptions.cs#L16
Non-nullable property 'BucketName' must contain a non-null value when exiting constructor. Consider adding the 'required' modifier or declaring the property as nullable.
|
|
build (windows-latest):
src/BaGetter.Tencent/TencentStorageService.cs#L29
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.
|