Linting and tests #947
mlapagliadocker-publish.yml
on: pull_request
build
1m 20s
back-end-tests
2m 22s
front-end-lint
21s
front-end-tests
25s
Annotations
10 warnings
|
back-end-tests:
OpenAlprWebhookProcessor.Server/Data/Migrations/20210202185642_agentlongitude.cs#L26
Define a constant instead of using this literal 'SunriseOffset' 4 times. (https://rules.sonarsource.com/csharp/RSPEC-1192)
|
|
back-end-tests:
OpenAlprWebhookProcessor.Server/Data/Migrations/20210202185642_agentlongitude.cs#L11
Define a constant instead of using this literal 'Cameras' 8 times. (https://rules.sonarsource.com/csharp/RSPEC-1192)
|
|
back-end-tests:
OpenAlprWebhookProcessor.Server/Data/Migrations/20210131210307_agentcoordinates.cs#L17
Define a constant instead of using this literal 'Agents' 4 times. (https://rules.sonarsource.com/csharp/RSPEC-1192)
|
|
back-end-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)
|
|
back-end-tests:
OpenAlprWebhookProcessor.Server/Data/Migrations/20210109164132_refactor.cs#L15
Define a constant instead of using this literal 'PlateGroups' 16 times. (https://rules.sonarsource.com/csharp/RSPEC-1192)
|
|
back-end-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)
|
|
back-end-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)
|
|
back-end-tests:
OpenAlprWebhookProcessor.Server/ImageRelay/GetImage/GetImageHandler.cs#L170
The variable 'ex' is declared but never used
|
|
back-end-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.
|
|
back-end-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.
|