feat(core): refactor file collector for more customization #378
alexseeAnnotations
21 warnings
|
build
Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: actions/setup-java@v3, actions/setup-dotnet@v3, actions/checkout@v3. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.
|
|
build:
src/BSH.Engine/Services/BackupService.cs#L68
Avoid using "DateTime.Now" for benchmarking or timespan calculation operations. (https://rules.sonarsource.com/csharp/RSPEC-6561)
|
|
build:
src/BSH.Engine/Services/FileCollector/IFolderExclusion.cs#L64
Use PascalCase for named placeholders. (https://rules.sonarsource.com/csharp/RSPEC-6678)
|
|
build:
src/BSH.Engine/Services/FileCollector/FileCollectorService.cs#L77
Collection-specific "Exists" method should be used instead of the "Any" extension. (https://rules.sonarsource.com/csharp/RSPEC-6605)
|
|
build:
src/BSH.Engine/Services/FileCollector/FileCollectorService.cs#L105
Collection-specific "Exists" method should be used instead of the "Any" extension. (https://rules.sonarsource.com/csharp/RSPEC-6605)
|
|
build:
src/BSH.Engine/Services/FileCollector/FileCollectorService.cs#L114
Use PascalCase for named placeholders. (https://rules.sonarsource.com/csharp/RSPEC-6678)
|
|
build:
src/BSH.Engine/Services/FileCollector/FileCollectorService.cs#L120
Use PascalCase for named placeholders. (https://rules.sonarsource.com/csharp/RSPEC-6678)
|
|
build:
src/BSH.Engine/Jobs/BackupJob.cs#L105
Refactor this method to reduce its Cognitive Complexity from 44 to the 15 allowed. (https://rules.sonarsource.com/csharp/RSPEC-3776)
|
|
build:
src/BSH.Engine/Jobs/BackupJob.cs#L216
Use PascalCase for named placeholders. (https://rules.sonarsource.com/csharp/RSPEC-6678)
|
|
build:
src/BSH.Engine/Jobs/BackupJob.cs#L295
Use PascalCase for named placeholders. (https://rules.sonarsource.com/csharp/RSPEC-6678)
|
|
build:
src/BSH.Engine/Jobs/BackupJob.cs#L299
Use PascalCase for named placeholders. (https://rules.sonarsource.com/csharp/RSPEC-6678)
|
|
build:
src/BSH.Service/VSS/XCopy.cs#L29
Cannot convert null literal to non-nullable reference type.
|
|
build:
src/BSH.Service/VSS/XCopy.cs#L43
Non-nullable event 'Completed' must contain a non-null value when exiting constructor. Consider declaring the event as nullable.
|
|
build:
src/BSH.Service/VSS/XCopy.cs#L43
Non-nullable event 'ProgressChanged' must contain a non-null value when exiting constructor. Consider declaring the event as nullable.
|
|
build:
src/BSH.Service/VSS/Snapshot.cs#L27
Non-nullable field '_props' must contain a non-null value when exiting constructor. Consider declaring the field as nullable.
|
|
build:
src/BSH.Service/VSS/VssBackup.cs#L38
Non-nullable field '_backup' must contain a non-null value when exiting constructor. Consider declaring the field as nullable.
|
|
build:
src/BSH.Service/VSS/VssBackup.cs#L38
Non-nullable field '_snap' must contain a non-null value when exiting constructor. Consider declaring the field as nullable.
|
|
build:
src/BSH.Service/VSS/VssBackup.cs#L70
Cannot convert null literal to non-nullable reference type.
|
|
build:
src/BSH.Service/VSS/VssBackup.cs#L76
Cannot convert null literal to non-nullable reference type.
|
|
build:
src/BSH.Service/VSSService.cs#L31
Dereference of a possibly null reference.
|
|
build:
src/BSH.Service/VSSService.cs#L22
Non-nullable field 'exception' must contain a non-null value when exiting constructor. Consider declaring the field as nullable.
|