Skip to content
CI-CD PIMS Development

psp-7034 add warning when interest holders attached to acquisition pr… #1146

Sign in to view logs

psp-7034 add warning when interest holders attached to acquisition pr…

psp-7034 add warning when interest holders attached to acquisition pr… #1146

Triggered via pull request October 30, 2023 18:13
@devinleighsmithdevinleighsmith
closed #3550
Status Success
Total duration 13m 54s
Artifacts –

ci-cd-pims-dev.yml

on: pull_request_target
CI-CD Start Notification to Teams Channel
5s
CI-CD Start Notification to Teams Channel
Build frontend
6m 35s
Build frontend
Build api
3m 37s
Build api
Deploy frontend and api to OpenShift
2m 14s
Deploy frontend and api to OpenShift
Upgrade database
48s
Upgrade database
Sync Keycloak
3m 5s
Sync Keycloak
CI-CD End Notification to Teams Channel
10s
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/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/entities/Partials/AcqChklstSectionType.cs#L23
Either remove or fill this block of code.
Sync Keycloak: source/backend/dal/Repositories/UserRepository.cs#L262
Extract this nested ternary operation into an independent statement.
Sync Keycloak: source/backend/dal/Repositories/UserRepository.cs#L263
Extract this nested ternary operation into an independent statement.
Sync Keycloak: source/backend/api/Areas/Admin/Controllers/ClaimController.cs#L1
Remove this unnecessary 'using'.