feat(core): refactor file collector for more customization (#299) #379
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.Service.Shared/IVSSRemoteObject.cs#L19
Rename interface 'IVSSRemoteObject' to match pascal case naming rules, consider using 'IVssRemoteObject'. (https://rules.sonarsource.com/csharp/RSPEC-101)
|
|
build:
src/BSH.Engine/Security/Encryption.cs#L22
Complete the task associated to this 'TODO' comment. (https://rules.sonarsource.com/csharp/RSPEC-1135)
|
|
build:
src/BSH.Engine/Jobs/DeleteJob.cs#L123
Define a constant instead of using this literal 'fileName' 4 times. (https://rules.sonarsource.com/csharp/RSPEC-1192)
|
|
build:
src/BSH.Engine/Jobs/DeleteSingleJob.cs#L101
Define a constant instead of using this literal 'fileName' 6 times. (https://rules.sonarsource.com/csharp/RSPEC-1192)
|
|
build:
src/BSH.Engine/Jobs/DeleteSingleJob.cs#L102
Define a constant instead of using this literal 'filePath' 7 times. (https://rules.sonarsource.com/csharp/RSPEC-1192)
|
|
build:
src/BSH.Engine/QueryManager.cs#L253
Define a constant instead of using this literal 'SELECT a.versionID FROM versiontable a ' 4 times. (https://rules.sonarsource.com/csharp/RSPEC-1192)
|
|
build:
src/BSH.Engine/QueryManager.cs#L255
Define a constant instead of using this literal 'AND a.versionStatus = 0 ' 4 times. (https://rules.sonarsource.com/csharp/RSPEC-1192)
|
|
build:
src/BSH.Engine/QueryManager.cs#L258
Define a constant instead of using this literal 'AND filelink.versionID = a.versionID ' 4 times. (https://rules.sonarsource.com/csharp/RSPEC-1192)
|
|
build:
src/BSH.Engine/QueryManager.cs#L259
Define a constant instead of using this literal 'AND filetable.fileID = fileversiontable.fileID ' 5 times. (https://rules.sonarsource.com/csharp/RSPEC-1192)
|
|
build:
src/BSH.Engine/Jobs/RestoreJob.cs#L206
Define a constant instead of using this literal 'fileName' 5 times. (https://rules.sonarsource.com/csharp/RSPEC-1192)
|
|
build:
src/BSH.Service.Shared/IVSSRemoteObject.cs#L19
Rename interface 'IVSSRemoteObject' to match pascal case naming rules, consider using 'IVssRemoteObject'. (https://rules.sonarsource.com/csharp/RSPEC-101)
|
|
build:
src/PreviewHandlerFramework/StreamBasedPreviewHandler.cs#L24
Private classes which are not derived in the current assembly should be marked as 'sealed'. (https://rules.sonarsource.com/csharp/RSPEC-3260)
|
|
build:
src/PreviewHandlerFramework/COMInterop/COLORREF.cs#L11
Rename struct 'COLORREF' to match pascal case naming rules, consider using 'Colorref'. (https://rules.sonarsource.com/csharp/RSPEC-101)
|
|
build:
src/PreviewHandlerFramework/COMInterop/LOGFONT.cs#L10
Rename class 'LOGFONT' to match pascal case naming rules, consider using 'Logfont'. (https://rules.sonarsource.com/csharp/RSPEC-101)
|
|
build:
src/PreviewHandlerFramework/COMInterop/MSG.cs#L13
Remove the unused private field 'hwnd'. (https://rules.sonarsource.com/csharp/RSPEC-1144)
|
|
build:
src/PreviewHandlerFramework/COMInterop/MSG.cs#L15
Remove the unused private field 'wParam'. (https://rules.sonarsource.com/csharp/RSPEC-1144)
|
|
build:
src/PreviewHandlerFramework/COMInterop/MSG.cs#L16
Remove the unused private field 'lParam'. (https://rules.sonarsource.com/csharp/RSPEC-1144)
|
|
build:
src/PreviewHandlerFramework/COMInterop/MSG.cs#L11
Rename struct 'MSG' to match pascal case naming rules, consider using 'Msg'. (https://rules.sonarsource.com/csharp/RSPEC-101)
|
|
build:
src/PreviewHandlerFramework/COMInterop/RECT.cs#L11
Rename struct 'RECT' to match pascal case naming rules, consider using 'Rect'. (https://rules.sonarsource.com/csharp/RSPEC-101)
|
|
build:
src/PreviewHandlerFramework/PreviewHandlerAttribute.cs#L12
Make '_name' 'readonly'. (https://rules.sonarsource.com/csharp/RSPEC-2933)
|