Skip to content

[CI] Rework docs deployment #77

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 8 commits into from
Nov 7, 2025
+15 −6
Merged

[CI] Rework docs deployment #77

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
8 commits
Select commit Hold shift + click to select a range
639a797
[CI] Rework docs deployment
giordano Nov 6, 2025
71ed6c0
[CI] Upload documentation build as an artifact
giordano Nov 6, 2025
83532ed
[docs] Push documentation to BreezeDocumentation
giordano Nov 6, 2025
42e881c
[CI] Use BreezeDocumentation Documenter key
giordano Nov 6, 2025
19132f8
[CI] Don't create token for PRs from forks
giordano Nov 7, 2025
907b6fa
[docs] Only push previews when all env vars are non-empty
giordano Nov 7, 2025
5f7621c
[CI] Don't use app to generate github token
giordano Nov 7, 2025
43a2ce4
Update link to documentation in README.md
giordano Nov 7, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
7 changes: 6 additions & 1 deletion .github/workflows/Documentation.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -51,7 +51,7 @@ jobs:
- uses: julia-actions/julia-docdeploy@v1
env:
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
DOCUMENTER_KEY: ${{ secrets.DOCUMENTER_KEY }}
DOCUMENTER_KEY: ${{ secrets.BREEZEDOCUMENTATION_DOCUMENTER_KEY }}
- name: Save Julia depot cache on cancel or failure
id: julia-cache-save
if: cancelled() || failure()
Expand All @@ -60,3 +60,8 @@ jobs:
path: |
${{ steps.julia-cache.outputs.cache-paths }}
key: ${{ steps.julia-cache.outputs.cache-key }}
- uses: actions/upload-artifact@v4
with:
name: documentation-build
path: docs/build
retention-days: 7
4 changes: 2 additions & 2 deletions README.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -5,11 +5,11 @@

<!-- description -->
<p align="center">
<strong>🌪 Fast and friendly Julia software for atmosphere simulations on CPUs and GPUs. https://numericalearth.github.io/Breeze.jl/dev/</strong>
<strong>🌪 Fast and friendly Julia software for atmosphere simulations on CPUs and GPUs. https://numericalearth.github.io/BreezeDocumentation/dev/</strong>
</p>

<p align="center">
<a href="https://numericalearth.github.io/Breeze.jl/dev/">
<a href="https://numericalearth.github.io/BreezeDocumentation/dev/">
<img alt="Documentation" src="https://img.shields.io/badge/documentation-in%20development-orange?style=flat-square">
</a>
<a href="https://github.com/NumericalEarth/Breeze.jl/discussions">
Expand Down
10 changes: 7 additions & 3 deletions docs/make.jl
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -29,9 +29,13 @@ makedocs(sitename="Breeze",
]
)

deploydocs(;
deploydocs(
;
repo = "github.com/NumericalEarth/Breeze.jl",
deploy_repo = "github.com/NumericalEarth/BreezeDocumentation",
devbranch = "main",
push_preview = true,
forcepush = true
# Only push previews if all the relevant environment variables are non-empty. This is an
# attempt to work around https://github.com/JuliaDocs/Documenter.jl/issues/2048.
push_preview = all(!isempty, (get(ENV, "GITHUB_TOKEN", ""), get(ENV, "DOCUMENTER_KEY", ""))),
Comment on lines +37 to +39
Copy link
Collaborator Author

@giordano giordano Nov 7, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'd need to wait for a PR by dependabot to be sure this is working correctly, but looking at https://github.com/NumericalEarth/Breeze.jl/actions/runs/19032077021/job/54348250773?pr=61#step:5:5 GITHUB_TOKEN isn't empty, but DOCUMENTER_KEY is, so making sure they're both non-empty should be a decent approach.

Copy link
Collaborator Author

@giordano giordano Nov 7, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I just learned about dependabot secrets, in principle we can set DOCUMENTER_KEY for dependabot too, but I think the solution I proposed should be robust in all cases, regardless of whether we decide to set DOCUMENTER_KEY for dependabot or not.

forcepush = true,
)
Loading