Repository navigation
ci(pages): skip deployment in unconfigured forks - #2615
ankitbuilds99 wants to merge 2 commits into
Conversation
What does this PR do
|
Thanks for opening this — it has been seen, and it is queued. This note is automated, but it is not a brush-off: it exists so you know where your PR stands instead of having to guess from silence. Current review status: working through a backlog. What that means for this PR, concretely:
Things that will genuinely speed it up whenever review does happen:
If this fixes a bug, a reproduction we can run is worth more than a description of the symptom. Thanks for contributing, and sorry in advance for the wait. |
|
@DeusData — this PR fixes the Pages CI failure observed in fork run 38325069400. It guards deployment to the canonical repository, so forks without Pages enabled no longer fail at configure-pages while upstream deployment remains unchanged. Please review and merge when convenient. Thank you! |
Summary
Fixes fork push CI when GitHub Pages is not configured.
The
Deploy Pagesworkflow currently runsactions/configure-pagesfor every push tomain. In forks without a Pages site, that action fails with:This adds a repository guard so deployment runs only in the canonical
DeusData/codebase-memory-mcprepository. The upstream Pages deployment remains unchanged; forks skip this repository-owned deployment instead of reporting a false CI failure.Observed in fork run
38325069400.Validation
.github/workflows/pages.yml.