Skip to content

Rename the default docs build directory to store/docs_build - #16

Merged
bonelifer merged 1 commit into
mainfrom
docs-build-store-path
Oct 7, 2026
Merged

bonelifer merged 1 commit into
mainfrom
docs-build-store-path

Conversation

@bonelifer

Copy link
Copy Markdown
Contributor

Renames the default docs build directory to store/docs_build.

  • Default: new installs build into store/docs_build.
  • Upgrades: new migration m9_docs_build_path moves boards still set to an earlier default (ext/phpbbmodders/documentation/docs-build or store/phpbbmodders_documentation) to store/docs_build when nothing is built at their current path. Boards with a build there keep it.
  • Fallback: the fallback from Read the build from store/ when the old default path is empty #15 now covers both earlier defaults. If the setting is one of them and empty, but store/docs_build has a build, that build is used, and the ACP notice names the configured path.
  • Docs: README, docs/securing-docs-build.md and the contrib/ deny snippets use the new path.

Testing
Claude added tests for the migration and the new fallback case (139 tests), which pass in Docker on PHP 8.2 and 8.4. On a 3.3.19 test board, a fresh install ends up on store/docs_build, and the phpbb-release-tools feature checks pass.


Investigated and written by Claude on behalf of William Jacoby (bonelifer).

New migration m9 moves boards still set to an earlier default
(ext/phpbbmodders/documentation/docs-build or
store/phpbbmodders_documentation) to store/docs_build when there is no
build at their current path; boards with a build there keep it. The
store/ fallback now covers both earlier defaults, and the ACP notice
names the configured path. README, the securing guide and the deny
snippets use the new path.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@bonelifer
bonelifer requested a review from kaileymsnay October 7, 2026 14:20
@bonelifer
bonelifer merged commit 594e819 into main Oct 7, 2026
7 checks passed
@bonelifer
bonelifer deleted the docs-build-store-path branch October 7, 2026 14:23
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