Update Doc Cleanup action to run monthly#2
Conversation
|
Thank you @eliascarv for improving our actions. We really need to take care of these doc builds, otherwise they pollute our git history. I will ask the help of @JoshuaLampert as he knows these actions in more depth. |
|
@JoshuaLampert to give you more context: we are trying to clean old docs every now and then to save storage on GitHub. If they consume all the storage, the docs build fails. We already save the docs in a separate repo to avoid blowing up the Meshes.jl repo with various commits to gh-pages with Makie.jl figures. |
| git config user.email "documenter@juliadocs.github.io" | ||
| git rm -rf v[0-9]*.[0-9]* && git checkout HEAD -- $keep | ||
| if git commit -m "keep latest docs only" ; then | ||
| git rm -rf --ignore-unmatch v[0-9]*.[0-9]* |
There was a problem hiding this comment.
What about the previews in https://github.com/JuliaGeometry/MeshesDocs/tree/92ed26afc2387e6a41083183b84fae2baeba7153/previews? Doesn't it make sense to delete them as well? We have a similar setup in https://github.com/trixi-framework/TrixiDocumentation/blob/7dbc9d48d4c008f98498e2f4fed85b7855bb0f38/.github/workflows/DocPreviewsCleanup.yml, where we delete the previews/ folder.
There was a problem hiding this comment.
@JoshuaLampert, I wanted to implement logic to delete previews only for PRs that have already been merged/closed, while keeping the previews for open PRs.
However, simply clearing the folder seems like a reasonable option to me.
What do you think, @juliohm?
|
I think we can cleanup all previews without issues. The preview is
something that we rarely check in practice.
Em ter., 23 de jun. de 2026, 15:13, Elias Carvalho ***@***.***>
escreveu:
… ***@***.**** commented on this pull request.
------------------------------
In .github/workflows/DocCleanup.yml
<#2 (comment)>
:
> git config user.name "Documenter.jl"
git config user.email ***@***.***"
- git rm -rf v[0-9]*.[0-9]* && git checkout HEAD -- $keep
- if git commit -m "keep latest docs only" ; then
+ git rm -rf --ignore-unmatch v[0-9]*.[0-9]*
@JoshuaLampert <https://github.com/JoshuaLampert>, I wanted to implement
logic to delete previews only for PRs that have already been merged/closed,
while keeping the previews for open PRs.
However, simply clearing the folder seems like a reasonable option to me.
What do you think, @juliohm <https://github.com/juliohm>?
—
Reply to this email directly, view it on GitHub
<#2?email_source=notifications&email_token=AAZQW3NFJZL5X37ZKW4VMXT5BLCGLA5CNFSNUABKM5UWIORPF5TWS5BNNB2WEL2QOVWGYUTFOF2WK43UKJSXM2LFO4XTINJVGU4TGNRRG442M4TFMFZW63VHNVSW45DJN5XKKZLWMVXHJLDGN5XXIZLSL5RWY2LDNM#discussion_r3461938094>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAZQW3LHQ52GZGZNZFCYKJL5BLCGLAVCNFSNUABFKJSXA33TNF2G64TZHM3DMOJVGY4TMOBQHNEXG43VMU5TINZSG43DKOBVGUZKC5QC>
.
Triage notifications, keep track of coding agent tasks and review pull
requests on the go with GitHub Mobile for iOS
<https://github.com/notifications/mobile/ios/AAZQW3M2GQPO27XAZVH76HL5BLCGLA5CNFSNUABKM5UWIORPF5TWS5BNNB2WEL2QOVWGYUTFOF2WK43UKJSXM2LFO4XTINJVGU4TGNRRG442M4TFMFZW63VHNVSW45DJN5XKKZLWMVXHJKTGN5XXIZLSL5UW64Y>
and Android
<https://github.com/notifications/mobile/android/AAZQW3IDHFMA6C4QLIDQYBL5BLCGLA5CNFSNUABKM5UWIORPF5TWS5BNNB2WEL2QOVWGYUTFOF2WK43UKJSXM2LFO4XTINJVGU4TGNRRG442M4TFMFZW63VHNVSW45DJN5XKKZLWMVXHJLTGN5XXIZLSL5QW4ZDSN5UWI>.
Download it today!
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
|
@juliohm great, I'll adjust the PR after running tests on a fork. |
|
Thank you @eliascarv, appreciate it if you could reproduce this new setting in the other PRs to GeoStatsDocs and TableTransforms.jl. That way we get the same behavior across projects. |
|
@JoshuaLampert ready for final review. |
3 participants
Update the action with the new features implemented in JuliaML/TableTransforms.jl#323.