Skip to content

Commit 62c8a6a

Browse files
committed
CCM-11938: Fix sonar issues
1 parent a12418d commit 62c8a6a

File tree

2 files changed

+14
-12
lines changed

2 files changed

+14
-12
lines changed

.github/actions/build-proxies/action.yml

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -81,6 +81,11 @@ runs:
8181
echo "Building env specific OAS spec"
8282
make build-json-oas-spec APIM_ENV=${{ inputs.apimEnv }}
8383
fi
84+
if [[ $APIM_ENV == *-pr ]]; then
85+
echo "Removing pr suffix from APIM_ENV after building OAS and calling proxygen"
86+
APIM_ENV=$(echo "$APIM_ENV" | sed 's/-pr$//')
87+
echo "APIM_ENV=$APIM_ENV" >> $GITHUB_ENV
88+
fi
8489
8590
- name: Upload OAS Spec
8691
uses: actions/upload-artifact@v4

.github/workflows/manual-proxy-environment-deploy.yaml

Lines changed: 9 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -70,18 +70,15 @@ jobs:
7070
7171
echo "ENVIRONMENT=$ENVIRONMENT" >> $GITHUB_ENV
7272
echo "APIM_ENV=$APIM_ENV" >> $GITHUB_ENV
73-
- name: Build environment oas
74-
working-directory: .
75-
env:
76-
APIM_ENV: ${{ env.APIM_ENV }}
77-
shell: bash
78-
run: |
79-
make build-json-oas-spec APIM_ENV=$APIM_ENV
80-
if [[ $APIM_ENV == *-pr ]]; then
81-
echo "Removing pr suffix from APIM_ENV after building OAS and calling proxygen"
82-
APIM_ENV=$(echo "$APIM_ENV" | sed 's/-pr$//')
83-
echo "APIM_ENV=$APIM_ENV" >> $GITHUB_ENV
84-
fi
73+
# - name: Build environment oas
74+
# working-directory: .
75+
# env:
76+
# APIM_ENV: ${{ env.APIM_ENV }}
77+
# shell: bash
78+
# run: |
79+
# make build-json-oas-spec APIM_ENV=$APIM_ENV
80+
81+
# fi
8582
- name: "Build proxies"
8683
env:
8784
PROXYGEN_API_NAME: nhs-notify-supplier

0 commit comments

Comments
 (0)