Skip to content

Change event trigger from pull_request_target to pull_request - #55112

Closed
gewarren wants to merge 1 commit into
mainfrom
gewarren-patch-8
Closed

Change event trigger from pull_request_target to pull_request#55112
gewarren wants to merge 1 commit into
mainfrom
gewarren-patch-8

Conversation

@gewarren

Copy link
Copy Markdown
Collaborator

No description provided.

@gewarren
gewarren requested a review from a team as a code owner July 29, 2026 02:34
Copilot AI review requested due to automatic review settings July 29, 2026 02:34
@dotnetrepoman dotnetrepoman Bot added this to the July 2026 milestone Jul 29, 2026
@gewarren
gewarren enabled auto-merge (squash) July 29, 2026 02:34

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

This PR changes the trigger for the “OPS status checker” GitHub Actions workflow from pull_request_target to pull_request, affecting when/how it runs on pull requests and what permissions/secrets are available during execution.

Changes:

  • Update workflow event trigger from pull_request_target to pull_request.

Comment on lines 3 to 5
on:
pull_request_target:
pull_request:
types: [opened, synchronize, reopened]

@wadepickett wadepickett 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.

@gewarren: Copilot raised issue needs to be resolved. I am not finding any other issue beyond that one though.

@gewarren

Copy link
Copy Markdown
Collaborator Author

This will prevent the action from running correctly as it needs access to a secret.

@gewarren gewarren closed this Jul 29, 2026
auto-merge was automatically disabled July 29, 2026 22:13

Pull request was closed

@gewarren
gewarren deleted the gewarren-patch-8 branch July 29, 2026 22:13
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