Skip to content

fix Trigger#28

Merged
ejfine merged 1 commit intomainfrom
trigger2
May 9, 2025
Merged

fix Trigger#28
ejfine merged 1 commit intomainfrom
trigger2

Conversation

@ejfine
Copy link
Copy Markdown
Contributor

@ejfine ejfine commented May 9, 2025

No description provided.

@ejfine ejfine requested a review from Copilot May 9, 2025 12:55
@ejfine ejfine self-assigned this May 9, 2025
@ejfine ejfine merged commit 9bd84a7 into main May 9, 2025
2 checks passed
Copy link
Copy Markdown

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 fixes the CI trigger by enabling manual workflow dispatch and updates the copier template commit reference.

  • Enabled manual trigger using workflow_dispatch for the CI workflow.
  • Updated the commit hash in the copier configuration file.

Reviewed Changes

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

File Description
.github/workflows/ci.yaml Added workflow_dispatch to enable manual triggering of the CI.
.copier-answers.yml Updated the copier template commit hash.

branches-ignore:
- 'gh-readonly-queue/**' # don't run (again) when on these special branches created during merge groups; the `on: merge_group` already triggers it.
merge_group:
workflow_dispatch:
Copy link

Copilot AI May 9, 2025

Choose a reason for hiding this comment

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

[nitpick] Consider adding an inline comment to describe why workflow_dispatch was added, to assist future maintainers in understanding its purpose in the CI trigger configuration.

Suggested change
workflow_dispatch:
workflow_dispatch: # Allows manual triggering of this workflow, useful for testing or debugging purposes.

Copilot uses AI. Check for mistakes.
@ejfine ejfine deleted the trigger2 branch May 9, 2025 17:53
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