Skip to content

Commit

Permalink
don't use Azure if the secrets are not available
Browse files Browse the repository at this point in the history
  • Loading branch information
mmomtchev committed Sep 27, 2024
1 parent abbf991 commit d434d16
Showing 1 changed file with 6 additions and 2 deletions.
8 changes: 6 additions & 2 deletions .github/workflows/windows-release.yml
Original file line number Diff line number Diff line change
Expand Up @@ -148,7 +148,9 @@ jobs:
nmake release
- name: 'Azure CLI login with federated credential'
if: github.event_name != 'pull_request'
env:
secrets_available: ${{ secrets.AZURE_SUBSCRIPTION_ID }}
if: ${{ github.event_name != 'pull_request' }} && ${{ !!env.secrets_available }}
uses: azure/#@v2
with:
client-id: ${{ secrets.AZURE_CLIENT_ID }}
Expand Down Expand Up @@ -325,7 +327,9 @@ jobs:
msbuild /m /t:Rebuild /p:Configuration=Release,Platform=${{matrix.platform}}
- name: 'Azure CLI login with federated credential'
if: github.event_name != 'pull_request'
env:
secrets_available: ${{ secrets.AZURE_SUBSCRIPTION_ID }}
if: ${{ github.event_name != 'pull_request' }} && ${{ !!env.secrets_available }}
uses: azure/#@v2
with:
client-id: ${{ secrets.AZURE_CLIENT_ID }}
Expand Down

0 comments on commit d434d16

Please # to comment.