Skip to content

Update the website deploy step to trigger when using workflow_dispatch - #261

Open
ruokun-niu wants to merge 1 commit into
drasi-project:mainfrom
ruokun-niu:main
Open

Update the website deploy step to trigger when using workflow_dispatch#261
ruokun-niu wants to merge 1 commit into
drasi-project:mainfrom
ruokun-niu:main

Conversation

@ruokun-niu

Copy link
Copy Markdown
Contributor

Description

This pull request updates the deployment workflow for the website to allow deployments to be triggered both by a direct push to the main branch and manually via the "workflow_dispatch" event.

Workflow improvements:

Type of change

  • This pull request fixes a bug in Drasi and has an approved issue (issue link required).
  • This pull request adds or changes features of Drasi and has an approved issue (issue link required).
  • This pull request is a minor refactor, code cleanup, test improvement, or other maintenance task and doesn't change the functionality of Drasi (issue link optional).

Fixes: #issue_number

Signed-off-by: Ruokun Niu <ruokunniu@microsoft.com>

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Updates the GitHub Pages deployment workflow so the deploy job runs both on pushes to main and on manual workflow_dispatch runs (while still restricting deployment to the main ref).

Changes:

  • Adjusted the deploy job if: condition to allow both push and workflow_dispatch events.
  • Kept the deployment restricted to refs/heads/main to prevent deploying from non-main manual dispatches.

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

@ruokun-niu ruokun-niu self-assigned this Jul 31, 2026
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.

3 participants