Skip to content

Commit 4922e20

Browse files
Copilotpamelafox
andauthored
Remove conditional Azure login steps and simplify to single Federated Credentials auth (#240)
* Initial plan * Remove conditional Azure login steps and simplify to single Federated Credentials auth Co-authored-by: pamelafox <297042+pamelafox@users.noreply.github.com> --------- Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: pamelafox <297042+pamelafox@users.noreply.github.com>
1 parent 6fae638 commit 4922e20

File tree

2 files changed

+0
-15
lines changed

2 files changed

+0
-15
lines changed

.github/workflows/azure-dev.yaml

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -49,7 +49,6 @@ jobs:
4949
node-version: 20
5050

5151
- name: Log in with Azure (Federated Credentials)
52-
if: ${{ env.AZURE_CLIENT_ID != '' }}
5352
run: |
5453
azd auth login `
5554
--client-id "$Env:AZURE_CLIENT_ID" `

.github/workflows/evaluate.yaml

Lines changed: 0 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -91,21 +91,7 @@ jobs:
9191
- name: Install azd
9292
uses: Azure/setup-azd@v2.1.0
9393

94-
- name: Login to Azure
95-
uses: azure/login@v2
96-
with:
97-
client-id: ${{ env.AZURE_CLIENT_ID }}
98-
tenant-id: ${{ env.AZURE_TENANT_ID }}
99-
subscription-id: ${{ env.AZURE_SUBSCRIPTION_ID }}
100-
101-
- name: Set az account
102-
uses: azure/CLI@v2
103-
with:
104-
inlineScript: |
105-
az account set --subscription ${{env.AZURE_SUBSCRIPTION_ID}}
106-
10794
- name: Log in with Azure (Federated Credentials)
108-
if: ${{ env.AZURE_CLIENT_ID != '' }}
10995
run: |
11096
azd auth login `
11197
--client-id "$Env:AZURE_CLIENT_ID" `

0 commit comments

Comments
 (0)