Skip to content

chore: change pull_request_target to pull_request in Danger workflow#9

Merged
nekrich merged 1 commit into
masterfrom
fix/pull_request_target-invocation
May 27, 2026
Merged

chore: change pull_request_target to pull_request in Danger workflow#9
nekrich merged 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
@nekrich nekrich merged commit e980690 into master May 27, 2026
51 of 54 checks passed
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