Skip to content
This repository has been archived by the owner on Jan 30, 2023. It is now read-only.

Commit

Permalink
Don't use alias for now due to various netlify bugs
Browse files Browse the repository at this point in the history
  • Loading branch information
tobiasdiez committed Feb 7, 2022
1 parent e0bac5f commit e7c8e46
Showing 1 changed file with 8 additions and 2 deletions.
10 changes: 8 additions & 2 deletions .github/workflows/doc-build.yml
Original file line number Diff line number Diff line change
Expand Up @@ -40,8 +40,14 @@ jobs:
- name: Deploy to Netlify
uses: netlify/actions/cli@master
with:
args: deploy --dir=docs --alias="${NETLIFY_ALIAS}"
with:
# We could use --alias="${NETLIFY_ALIAS}" to fix the deploy url to the branch name.
# However, netlify currently doesn't support updates to a deploy with alias
# https://github.com/netlify/cli/issues/948
# https://github.com/netlify/cli/issues/1984
# Note that even if this feature is implemented, one would also need to first process the branch name
# to workaround the bug https://github.com/netlify/cli/issues/969.
args: deploy --dir=docs
env:
NETLIFY_ALIAS: ${{ github.ref_name }}
NETLIFY_AUTH_TOKEN: ${{ secrets.NETLIFY_AUTH_TOKEN }}
Expand Down

0 comments on commit e7c8e46

Please sign in to comment.