Skip to content

chore: change pull_request_target to pull_request in Danger workflow#9

Open
art-tykh wants to merge 1 commit into
masterfrom
fix/pull_request_target-invocation
Open

chore: change pull_request_target to pull_request in Danger workflow#9
art-tykh wants to merge 1 commit into
masterfrom
fix/pull_request_target-invocation

Conversation

@art-tykh
Copy link
Copy Markdown

New Pull Request Checklist


  • I have updated this branch with the latest master to avoid conflicts (via merge from master or rebase)
  • I have added the required tests to prove the fix/feature I am adding
  • I have updated the documentation (if necessary)
  • I have run the tests and they pass
  • I have run the lint and it passes (pod lib lint)

This merge request fixes / refers to the following issues: ...

Pull Request Description

...

Copilot AI review requested due to automatic review settings May 20, 2026 11:02
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

Updates the GitHub Actions trigger for the Danger workflow to run on pull_request events instead of pull_request_target, affecting how/when Danger executes and what permissions/token context it runs under.

Changes:

  • Change Danger workflow trigger from pull_request_target to pull_request.

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

Comment thread .github/workflows/danger.yml
@art-tykh art-tykh requested a review from nekrich May 20, 2026 11:07
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