Repository navigation
docs: exclude tooling files from the published docs site - #8770
Merged
Merged
Conversation
The docs build copies every non-Markdown file in docs_dir into the site, so the local preview Dockerfile and the docs requirements files were published at the root of every docs version. Exclude them from the build. Fixes #8769
powertools-for-aws-oss-automation
Bot
added
the
size/XS
dreamorosi
requested review from
leandrodamascena
and removed request for
hjgraca
November 4, 2026 17:28
leandrodamascena
approved these changes
Nov 5, 2026
leandrodamascena
left a comment
leandrodamascena
left a comment
Contributor
There was a problem hiding this comment.
Thanks Andrea! I checked and the Dockerfile is indeed live on the site right now, with your change it's gone from the build. Approving.
boring-cyborg
Bot
added
the
documentation
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Issue number: closes #8769
Summary
Changes
Add
/Dockerfileand/requirements.*toexclude_docsinmkdocs.yml, so the local docs preview container and the docs dependency files stop being copied into the published site.User experience
Before:
Dockerfile,requirements.inandrequirements.txtare reachable at the root of every docs version, e.g. https://docs.aws.amazon.com/powertools/python/latest/Dockerfile.After: newly published versions don't include them. Pages and navigation are unchanged.
Verification
Ran
zensical build(0.0.73, fromdocs/requirements.txt) ondevelopbefore and after the change. Before: the three files appear at the root ofsite/. After: they're gone,adr/is still excluded, and the build reports no issues.This PR was prepared with an AI coding assistant. I reviewed the change and the build output.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.
Disclaimer: We value your time and bandwidth. As such, any pull requests created on non-triaged issues might not be successful.