Skip to content

馃懛 Disable saving Zensical's .cache in build-docs.yml - #16156

Merged
tiangolo merged 1 commit into
masterfrom
remove-actions/cache-for-zensical-cache
Aug 7, 2026
Merged

tiangolo merged 1 commit into
masterfrom
remove-actions/cache-for-zensical-cache

Conversation

@YuriiMotov

@YuriiMotov YuriiMotov commented Aug 7, 2026 •

Copy link
Copy Markdown
Member

Description

Disable saving Zensical's .cache in build-docs.yml.
It's currently useless because we remove directory with .cache before running zensical build in scripts/docs.py (see code).

I attempted to fix it by modifying the script to not delete .cache, but after some investigation I realized that it's fragile - Zensical's cache invalidation logic doesn't take into account docs_src or .yaml-files in data. So, it will generate outdated content if we change them.

AI Disclaimer

Used Claude Code (Opus 5) to investigate and confirm the solution

Checklist

  • This PR links to a GitHub Discussion for the proposed code change.
  • I added tests for the change.
  • The new or updated tests fail on the main branch and pass on this PR.
  • Coverage stays at 100%.
  • The documentation explains the change if needed.

@YuriiMotov YuriiMotov added the internal Internal changes label Aug 7, 2026
@github-actions

github-actions Bot commented Aug 7, 2026

Copy link
Copy Markdown
Contributor

馃摑 Docs preview

Last commit 9733d42 at: https://ffac0e56.fastapitiangolo.pages.dev

@tiangolo tiangolo left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thank you! 馃檶

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

internal Internal changes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants