Skip to content

Automation Refactoring #1195

Automation Refactoring

Automation Refactoring #1195

Triggered via pull request November 17, 2023 00:02
@asanchezrasanchezr
closed #3598
Status Success
Total duration 13m 0s
Artifacts

ci-cd-pims-dev.yml

on: pull_request_target
CI-CD Start Notification to Teams Channel
3s
CI-CD Start Notification to Teams Channel
Deploy frontend and api to OpenShift
2m 20s
Deploy frontend and api to OpenShift
Upgrade database
1m 0s
Upgrade database
CI-CD End Notification to Teams Channel
4s
CI-CD End Notification to Teams Channel
Fit to window
Zoom out
Zoom in

Annotations

10 warnings
Sync Keycloak: source/backend/core/Extensions/QueryableExtensions.cs#L17
Make sure that this accessibility bypass is safe here.
Sync Keycloak: source/backend/core/Json/NullJsonResultFilter.cs#L19
Add a nested comment explaining why this method is empty, throw a 'NotSupportedException' or complete the implementation.
Sync Keycloak: source/backend/core/Extensions/DictionaryExtensions.cs#L70
Extract this nested ternary operation into an independent statement.
Sync Keycloak: source/backend/core/Extensions/DictionaryExtensions.cs#L106
Extract this nested ternary operation into an independent statement.
Sync Keycloak: source/backend/entities/Partials/AcqChklstSectionType.cs#L23
Either remove or fill this block of code.
Sync Keycloak: source/backend/entities/Partials/AcqChklstItemType.cs#L29
Either remove or fill this block of code.
Sync Keycloak: source/backend/entities/Partials/AcqChklstItemType.cs#L29
Use the 'value' parameter in this property set accessor declaration.
Sync Keycloak: source/backend/dal/Exceptions/OverrideExceptions.cs#L40
Braces for multi-line statements should not share line
Sync Keycloak: source/backend/dal/Repositories/ProductRepository.cs#L4
Remove this unnecessary 'using'.
Sync Keycloak: source/backend/dal/Repositories/ProjectRepository.cs#L1
Remove this unnecessary 'using'.