Skip to content

Conversation

@FanwangM
Copy link
Contributor

@FanwangM FanwangM commented Dec 7, 2025

No description provided.

Copilot AI review requested due to automatic review settings December 7, 2025 17:05
Copy link

Copilot AI left a comment

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 enables the website building workflow to trigger on version release tags in addition to pushes to the main branch. The workflow will now build and deploy documentation when semantic version tags are pushed to the repository.

Key Changes:

  • Added tag patterns to trigger workflow on version releases (e.g., v1.0.0, 1.0.0)
  • Included support for pre-release tags (e.g., v1.0.0-alpha.1)

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

Comment on lines +9 to +13
- "v[0-9].[0-9].[0-9]*"
- "[0-9].[0-9].[0-9]*"
# Trigger on pre-release tags (e.g., v1.0.0-alpha.1)
- "v[0-9].[0-9].[0-9]*-*"
- "[0-9].[0-9].[0-9]*-*"
Copy link

Copilot AI Dec 7, 2025

Choose a reason for hiding this comment

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

The tag patterns will only match versions where the major and minor version numbers are single digits (0-9). For example, v10.0.0 or v1.15.0 won't trigger the workflow.

Consider updating the patterns to support multi-digit version numbers:

- "v[0-9]+.[0-9]+.[0-9]+"
- "[0-9]+.[0-9]+.[0-9]+"
# For pre-release tags
- "v[0-9]+.[0-9]+.[0-9]+-*"
- "[0-9]+.[0-9]+.[0-9]+-*"

This ensures all valid semantic version tags will trigger the workflow, not just those with single-digit major and minor versions.

Suggested change
- "v[0-9].[0-9].[0-9]*"
- "[0-9].[0-9].[0-9]*"
# Trigger on pre-release tags (e.g., v1.0.0-alpha.1)
- "v[0-9].[0-9].[0-9]*-*"
- "[0-9].[0-9].[0-9]*-*"
- "v[0-9]+.[0-9]+.[0-9]+"
- "[0-9]+.[0-9]+.[0-9]+"
# Trigger on pre-release tags (e.g., v1.0.0-alpha.1)
- "v[0-9]+.[0-9]+.[0-9]+-*"
- "[0-9]+.[0-9]+.[0-9]+-*"

Copilot uses AI. Check for mistakes.
@FanwangM FanwangM closed this Dec 19, 2025
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