Skip to content

Conversation

@ChronosSF
Copy link
Member

No description provided.

@ChronosSF ChronosSF requested a review from Copilot October 31, 2025 16:44
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 adds a new GitHub Actions workflow to trigger deployment in a separate repository. The workflow responds to pushes on vnext and master branches or manual dispatch, and uses the GitHub API to send a repository dispatch event to the igniteui-actions repository.

Key Changes:

  • New workflow file that triggers on push to vnext/master branches and manual workflow dispatch
  • Uses GitHub API to send repository dispatch events with branch information and test configuration
  • Configured with read-only content permissions

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

@ChronosSF ChronosSF requested a review from Copilot October 31, 2025 16:53
@ChronosSF ChronosSF merged commit 3d8cb55 into master Oct 31, 2025
13 of 14 checks passed
@ChronosSF ChronosSF deleted the sstoychev/cd-trigger-master branch October 31, 2025 16:54
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

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


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

curl -L \
-X POST \
-H "Accept: application/vnd.github+json" \
-H "Authorization: Bearer ${{ secrets.FINE_PATKA }}" \
Copy link

Copilot AI Oct 31, 2025

Choose a reason for hiding this comment

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

The secret name 'FINE_PATKA' appears to be a typo. It should likely be 'FINE_PAT_KA' or 'FINE_GRAINED_PAT' based on standard naming conventions for fine-grained personal access tokens.

Suggested change
-H "Authorization: Bearer ${{ secrets.FINE_PATKA }}" \
-H "Authorization: Bearer ${{ secrets.FINE_GRAINED_PAT }}" \

Copilot uses AI. Check for mistakes.
repo_owner="IgniteUI"
repo_name="igniteui-actions"
event_type="igniteui-angular-samples-cd"
branch="${{ github.ref_name }}"
Copy link

Copilot AI Oct 31, 2025

Choose a reason for hiding this comment

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

The workflow_dispatch input 'branch' is not being used. When triggered manually via workflow_dispatch, the branch variable should use github.event.inputs.branch instead of always using github.ref_name.

Suggested change
branch="${{ github.ref_name }}"
branch="${{ github.event.inputs.branch || github.ref_name }}"

Copilot uses AI. Check for mistakes.
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