Skip to content

fix: preserve hidden Pages files#16

Merged
steipete merged 1 commit into
mainfrom
codex/pages-hidden-files
Jul 1, 2026
Merged

fix: preserve hidden Pages files#16
steipete merged 1 commit into
mainfrom
codex/pages-hidden-files

Conversation

@steipete

@steipete steipete commented Jul 1, 2026

Copy link
Copy Markdown
Collaborator

Summary

Root cause

upload-pages-artifact@v5 defaults include-hidden-files to false. The artifact from the merged Pages run omitted .nojekyll, confirmed by downloading and listing artifact 8006483649.

Proof

  • Upstream v5 action.yml confirms the default and exclusion behavior.
  • go run github.com/rhysd/actionlint/cmd/[email protected]
  • Local archive of docs contains ./.nojekyll.
  • Autoreview: clean; no accepted/actionable findings.

Risk

Low: one documented workflow input. The current docs tree contains only one hidden file, .nojekyll.

@steipete
steipete merged commit b3fba02 into main Jul 1, 2026
4 checks passed
@steipete
steipete deleted the codex/pages-hidden-files branch July 1, 2026 10:07
@steipete

steipete commented Jul 1, 2026

Copy link
Copy Markdown
Collaborator Author

Landed as b3fba02.

Exact-head proof for 664d202:

Post-merge proof for b3fba02:

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant