Skip to content

feat: Added filter to CD#74

Merged
KittyChiu merged 6 commits into
mainfrom
feature/add-cd-filter
May 13, 2026
Merged

feat: Added filter to CD#74
KittyChiu merged 6 commits into
mainfrom
feature/add-cd-filter

Conversation

@KittyChiu
Copy link
Copy Markdown
Collaborator

No description provided.

Copilot AI review requested due to automatic review settings May 6, 2026 01:20
@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 6, 2026

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

Scanned Files

None

Copy link
Copy Markdown
Contributor

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 adds a pre-check to the deployment workflow so it only continues when an approved PR changes files outside .github/workflows/. In this repository, that narrows when the “Merge Contribution to Internal” automation proceeds after review approval.

Changes:

  • Added a new check-file-changes job that inspects the PR’s changed file paths via gh pr view.
  • Introduced a should_run output to skip the rest of the workflow when all changes are under .github/workflows/.
  • Wired check-merge-state to depend on that filter result before continuing.
Show a summary per file
File Description
.github/workflows/deploy-merged-pr.yml Adds a path-based gate before the existing merge-state / deployment flow.

Copilot's findings

Tip

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

  • Files reviewed: 1/1 changed files
  • Comments generated: 1

Comment thread .github/workflows/deploy-merged-pr.yml Outdated
Copy link
Copy Markdown
Contributor

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.

Copilot's findings

Tip

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

  • Files reviewed: 1/1 changed files
  • Comments generated: 0 new

@KittyChiu KittyChiu enabled auto-merge (squash) May 6, 2026 01:35
Copy link
Copy Markdown
Contributor

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.

Copilot's findings

Tip

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

  • Files reviewed: 1/1 changed files
  • Comments generated: 0 new

@KittyChiu KittyChiu disabled auto-merge May 13, 2026 00:19
@KittyChiu KittyChiu merged commit 528ff57 into main May 13, 2026
6 checks passed
@KittyChiu KittyChiu deleted the feature/add-cd-filter branch May 13, 2026 02:37
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