Skip to content

Conversation

@bsbodden
Copy link
Collaborator

@bsbodden bsbodden commented Sep 5, 2025

…ules

The docs workflow was failing when triggered by release events because GitHub Pages environment protection rules block deployments from tags. This fix ensures docs are always built from the main branch while maintaining correct version information.

Changes:

  • Remove direct release event trigger to prevent duplicate workflow runs
  • Always checkout main branch to satisfy environment protection rules
  • Enhance version detection to use release tag when available
  • Rely on repository_dispatch from release workflow for docs deployment

The release workflow already triggers docs via repository_dispatch, making the direct release trigger redundant and problematic. This change creates a single, reliable docs deployment path that works with GitHub Pages protection rules.

Resolves deployment failures: "Tag 'v1.0.1' is not allowed to deploy to github-pages"

…ules

The docs workflow was failing when triggered by release events because GitHub Pages
environment protection rules block deployments from tags. This fix ensures docs are
always built from the main branch while maintaining correct version information.

Changes:
- Remove direct release event trigger to prevent duplicate workflow runs
- Always checkout main branch to satisfy environment protection rules
- Enhance version detection to use release tag when available
- Rely on repository_dispatch from release workflow for docs deployment

The release workflow already triggers docs via repository_dispatch, making the
direct release trigger redundant and problematic. This change creates a single,
reliable docs deployment path that works with GitHub Pages protection rules.

Resolves deployment failures: "Tag 'v1.0.1' is not allowed to deploy to github-pages"
@bsbodden bsbodden requested a review from jruaux September 5, 2025 23:56
@bsbodden bsbodden self-assigned this Sep 5, 2025
@bsbodden bsbodden merged commit c451ca9 into redis:main Sep 8, 2025
1 check passed
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