Skip to content

Commit

Permalink
ci: Run doc on PRs, just don't deploy them (#345)
Browse files Browse the repository at this point in the history
Issues like b7a9571 will be caught
before merging if we also attempt to build docs for PRs.

Just don't deploy them until they're merged.
  • Loading branch information
abhinav authored Aug 14, 2024
1 parent b7a9571 commit f7f31c9
Showing 1 changed file with 5 additions and 0 deletions.
5 changes: 5 additions & 0 deletions .github/workflows/doc.yml
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,8 @@ name: GitHub Pages
on:
push:
branches: [main]
pull_request:
branches: [ '*' ]
workflow_dispatch:

concurrency:
Expand Down Expand Up @@ -52,6 +54,9 @@ jobs:
deploy:
needs: build

# Deploy only on main builds, or for manual triggers.
if: github.event_name == 'push' || github.event_name == 'workflow_dispatch'

permissions:
pages: write
id-token: write
Expand Down

0 comments on commit f7f31c9

Please sign in to comment.