feedback fixes #12
DanielJurcaBSAnnotations
11 warnings
|
Build and Test
Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: actions/checkout@v3, actions/setup-dotnet@v3. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.
|
|
Build and Test:
src/Kentico.Xperience.Zapier/Admin/InfoModels/ApiKeyInfo.generated.cs#L29
#warning: '"You will need to configure the type info."'
|
|
Build and Test:
src/Kentico.Xperience.Zapier/Admin/InfoModels/ZapierTriggerInfo.generated.cs#L30
#warning: '"You will need to configure the type info."'
|
|
Build and Test:
src/Kentico.Xperience.Zapier/Actions/SelectorController.cs#L35
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 and Test:
src/Kentico.Xperience.Zapier/ZapierConfigurationController.cs#L82
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 and Test:
src/Kentico.Xperience.Zapier/Actions/ActionFormInsertController.cs#L86
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 and Test:
src/Kentico.Xperience.Zapier/Actions/ActionFormInsertController.cs#L104
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 and Test:
src/Kentico.Xperience.Zapier/Admin/ZapierModuleInstaller.cs#L56
Remove this commented out code. (https://rules.sonarsource.com/csharp/RSPEC-125)
|
|
Build and Test:
src/Kentico.Xperience.Zapier/Admin/ZapierModuleInstaller.cs#L155
Remove this commented out code. (https://rules.sonarsource.com/csharp/RSPEC-125)
|
|
Build and Test:
src/Kentico.Xperience.Zapier/Actions/SelectorController.cs#L20
Remove this unread private field 'logger' or refactor the code to use its value. (https://rules.sonarsource.com/csharp/RSPEC-4487)
|
|
Build and Test:
src/Kentico.Xperience.Zapier/ZapierConfigurationController.cs#L38
Use an immutable collection or reduce the accessibility of the non-private readonly field 'WorkflowEvents'. (https://rules.sonarsource.com/csharp/RSPEC-3887)
|