Skip to content

Fix actions for deploying documentation - #13

Merged
ghanse merged 3 commits into
mainfrom
fix_docs_publish
Jul 21, 2026
Merged

Fix actions for deploying documentation#13
ghanse merged 3 commits into
mainfrom
fix_docs_publish

Conversation

@ghanse

@ghanse ghanse commented Jul 21, 2026

Copy link
Copy Markdown
Collaborator

Changes

This PR fixes GitHub actions for deploying flowx documentation to GitHub pages.

Linked issues

N/A

Tests

  • manually tested
  • added unit tests
  • added integration tests

Copilot AI review requested due to automatic review settings July 21, 2026 15:53

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR updates GitHub Actions workflows to run CI and documentation build/deploy jobs on a protected runner group, supporting the goal of reliably deploying flowx documentation to GitHub Pages.

Changes:

  • Switch CI workflow jobs (not-a-fork, ci, fmt) to use databricks-solutions-protected-runner-group.
  • Switch docs build and GitHub Pages deploy jobs to use the same protected runner group.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
.github/workflows/push.yml Moves CI jobs to a protected runner group; includes a gating job for non-fork PRs.
.github/workflows/docs-release.yml Moves docs build and Pages deploy jobs to a protected runner group.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

runs-on:
group: databricks-solutions-protected-runner-group
labels: linux-ubuntu-latest
if: github.event_name == 'pull_request' && !github.event.pull_request.draft && !github.event.pull_request.head.repo.fork
@ghanse
ghanse merged commit 9fc3f6e into main Jul 21, 2026
3 checks passed
@ghanse
ghanse deleted the fix_docs_publish branch July 21, 2026 16:07
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.

2 participants