-
-
Notifications
You must be signed in to change notification settings - Fork 18
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.
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
Migrate the PDF build to Buildkite #26
base: master
Are you sure you want to change the base?
Conversation
070a6ea
to
f3b810b
Compare
Thanks for doing that. Very much look forward to not having someone complain about the broken PDF link for the first time in the past seven years. |
f3b810b
to
0b24f09
Compare
@DilumAluthge Buildkite is not yet deploying the PDF though, right? Or did I miss something? |
Ah you're right, I forgot we hadn't implemented the upload step yet. What are the commands that need to be run for the upload? (Assuming that we have the PDF file.) |
This is the function that handle the actual push to the docs.julialang.org/pdf/make.jl Lines 140 to 190 in 16a515e
Basically, add/update the PDFs to the branch, and then |
We also need to rename the PDF file, right? Currently, the PDF file being uploaded as a Buildkite artifact is just named |
Sorry for the delay. But if I'm not mistaken, Documenter already sets the name, and we just copy any PDF there is: https://github.com/JuliaDocs/Documenter.jl/blob/998d26e568a20e3b1764d0c2cef30ebe4d2563ae/src/Writers/LaTeXWriter.jl#L167-L174 |
Hmmm. I'm looking at a recent run on master (https://buildkite.com/julialang/julia-master/builds/17280#0183fada-b0ed-4d00-ab04-1d752f70514d), and in the artifacts tab, the PDF file is just named |
Oh, sorry, you are right. The renaming logic works by passing this docs.julialang.org/pdf/make.jl Line 74 in 16a515e
which has the version number, to this docs.julialang.org/pdf/make.jl Lines 58 to 69 in 16a515e
where we look for any PDF and rename it to Although I guess a more robust solution would be to handle it in |
The Buildkite equivalent is |
See also JuliaCI/julia-buildkite#198