Skip to content

Commit 6cc5b23

Browse files
committed
Switch Azure login logic
Signed-off-by: Keith Battocchi <[email protected]>
1 parent 27d3101 commit 6cc5b23

File tree

1 file changed

+20
-6
lines changed

1 file changed

+20
-6
lines changed

.github/workflows/publish-documentation.yml

Lines changed: 20 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -62,6 +62,7 @@ jobs:
6262
create_docs:
6363
name: Create and publish documentation
6464
runs-on: ubuntu-latest
65+
6566
steps:
6667
- uses: actions/checkout@v4
6768
name: Checkout repository
@@ -87,20 +88,33 @@ jobs:
8788
- run: python setup.py build_sphinx -b doctest
8889
name: Run doctests
8990
if : ${{ inputs.run_doctests }}
91+
92+
publish-docs:
93+
name: Publish documentation
94+
runs-on: ubuntu-latest
95+
needs: create_docs
96+
permissions:
97+
id-token: write # needed to publish to Azure
98+
if: ${{ inputs.publish }}
99+
steps:
100+
- uses: actions/download-artifact@v3
101+
name: Download docs artifact
102+
with:
103+
name: docs
104+
path: html
90105
- run: |-
91-
pushd build/sphinx/html
106+
pushd html
92107
zip -r docs.zip *
93108
popd
94109
name: Zip docs for publishing
95-
if : ${{ inputs.publish }}
96110
- uses: azure/login@v1
97111
name: Login to Azure
98112
with:
99-
creds: ${{ secrets.AZURE_CREDENTIALS }}
100-
if : ${{ inputs.publish }}
113+
client-id: ${{ secrets.AZURE_CLIENT_ID }}
114+
tenant-id: ${{ secrets.AZURE_TENANT_ID }}
115+
subscription-id: ${{ secrets.AZURE_SUBSCRIPTION_ID }}
101116
- uses: azure/webapps-deploy@v2
102117
name: Deploy documentation to Azure web app
103118
with:
104119
app-name: ${{ inputs.environment == 'prod' && 'econml' || 'econml-dev' }}
105-
package: build/sphinx/html/docs.zip
106-
if: ${{ inputs.publish }}
120+
package: html/docs.zip

0 commit comments

Comments
 (0)