Linting and tests #952
mlapagliadocker-publish.yml
on: pull_request
build
1m 16s
tests
2m 27s
front-end-lint
20s
Annotations
10 warnings
|
tests:
OpenAlprWebhookProcessor.Server/Cameras/Hikvision/HikvisionCamera.cs#L37
Define a constant instead of using this literal 'false' 4 times. (https://rules.sonarsource.com/csharp/RSPEC-1192)
|
|
tests:
OpenAlprWebhookProcessor.Server/LicensePlates/GetPlateFilters/GetLicensePlateFiltersHandler.cs#L18
Remove this unread private field '_logger' or refactor the code to use its value. (https://rules.sonarsource.com/csharp/RSPEC-4487)
|
|
tests:
OpenAlprWebhookProcessor.Server/LicensePlates/Enricher/LicensePlateData/LicensePlateDataClient.cs#L17
Refactor your code not to use hardcoded absolute paths or URIs. (https://rules.sonarsource.com/csharp/RSPEC-1075)
|
|
tests:
OpenAlprWebhookProcessor.Server/Program.cs#L7
Add a 'protected' constructor or the 'static' keyword to the class declaration. (https://rules.sonarsource.com/csharp/RSPEC-1118)
|
|
tests:
OpenAlprWebhookProcessor.Server/LicensePlates/GetMostSeenPlates/GetMostSeenPlatesRequest.cs#L5
Remove this empty class, write its code or make it an "interface". (https://rules.sonarsource.com/csharp/RSPEC-2094)
|
|
tests:
OpenAlprWebhookProcessor.Server/Alerts/Pushover/PushoverClient.cs#L17
Refactor your code not to use hardcoded absolute paths or URIs. (https://rules.sonarsource.com/csharp/RSPEC-1075)
|
|
tests:
OpenAlprWebhookProcessor.Server/Alerts/Pushover/PushoverClient.cs#L15
Refactor your code not to use hardcoded absolute paths or URIs. (https://rules.sonarsource.com/csharp/RSPEC-1075)
|
|
tests:
OpenAlprWebhookProcessor.Server/WebhookProcessor/OpenAlprAgentScraper/OpenAlprAgentScraper.cs#L172
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.
|
|
tests:
OpenAlprWebhookProcessor.Server/ImageRelay/GetImage/GetImageHandler.cs#L170
The variable 'ex' is declared but never used
|
|
tests:
OpenAlprWebhookProcessor.Server/WebPushSubscriptions/WebPushNotificationProducer.cs#L55
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.
|