Skip to content

Commit

Permalink
Merge pull request #466 from ably/docs-ci-external-contribs
Browse files Browse the repository at this point in the history
dev: dont run docs upload step on forks
  • Loading branch information
AndyTWF authored Feb 10, 2025
2 parents cb7299c + b29e024 commit b9616be
Showing 1 changed file with 5 additions and 1 deletion.
6 changes: 5 additions & 1 deletion .github/workflows/docs.yml
Original file line number Diff line number Diff line change
Expand Up @@ -33,13 +33,17 @@ jobs:

- name: Configure AWS Credentials
uses: aws-actions/configure-aws-credentials@v1
# Only run this step if it's a PR from the main repository, as forks won't have the necessary secrets
if: ${{ github.event.pull_request.head.repo.full_name == github.repository }}
with:
aws-region: eu-west-2
role-to-assume: arn:aws:iam::${{ secrets.ABLY_AWS_ACCOUNT_ID_SDK }}:role/ably-sdk-builds-ably-chat-js
role-session-name: "${{ github.run_id }}-${{ github.run_number }}"
role-session-name: '${{ github.run_id }}-${{ github.run_number }}'

- name: Upload Documentation
uses: ably/sdk-upload-action@v2
# Only run this step if it's a PR from the main repository, as forks won't have the necessary secrets
if: ${{ github.event.pull_request.head.repo.full_name == github.repository }}
with:
sourcePath: typedoc/generated
githubToken: ${{ secrets.GITHUB_TOKEN }}
Expand Down

0 comments on commit b9616be

Please sign in to comment.