ci(action-semantic-pull-request): Fix for PRs from forks#941
ci(action-semantic-pull-request): Fix for PRs from forks#941MaxymVlasov merged 4 commits intomasterfrom
Conversation
|
Note Other AI code review bot(s) detectedCodeRabbit has detected other AI code review bot(s) in this pull request and will avoid duplicating their findings in the review comments. This may lead to a less comprehensive review. 📝 WalkthroughSummary by CodeRabbit
WalkthroughSwitched GitHub Actions workflow trigger from Changes
Sequence Diagram(s)sequenceDiagram
autonumber
actor Dev as Developer
participant GH as GitHub
participant WF as pr-title Workflow
Dev->>GH: Open / Edit / Synchronize / Reopen PR
Note right of GH #DDDDFF: Events: opened, edited, synchronize, reopened
GH-->>WF: emit pull_request_target
activate WF
WF->>WF: run title validation jobs/steps
WF-->>GH: post status / checks
deactivate WF
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Pre-merge checks and finishing touches❌ Failed checks (1 warning)
✅ Passed checks (2 passed)
✨ Finishing touches🧪 Generate unit tests (beta)
📜 Recent review detailsConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro 📒 Files selected for processing (1)
🚧 Files skipped from review as they are similar to previous changes (1)
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (8)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
30aaa0d to
611dc08
Compare
There was a problem hiding this comment.
Actionable comments posted: 1
📜 Review details
Configuration used: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (1)
.github/workflows/pr-title.yml(1 hunks)
🧰 Additional context used
🪛 GitHub Actions: Common issues check
.github/workflows/pr-title.yml
[error] 3-9: Zizmor audit failure: dangerous workflow trigger detected (pull_request_target is insecure). Use of pull_request_target is almost always insecure; review workflow triggers.
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (13)
- GitHub Check: 🧹 Linters / metadata-validation@🐍3.13@ubuntu-latest
- GitHub Check: 🧪 Tests / pytest@🐍3.11@windows-2025
- GitHub Check: 🧪 Tests / pytest@🐍3.10@ubuntu-24.04
- GitHub Check: 🧹 Linters / pre-commit@🐍3.13@ubuntu-latest
- GitHub Check: 🧪 Tests / pytest@🐍3.12@windows-2025
- GitHub Check: 🧪 Tests / pytest@🐍3.11@ubuntu-24.04
- GitHub Check: 🧪 Tests / pytest@🐍3.12@ubuntu-24.04
- GitHub Check: 🧪 Tests / pytest@🐍3.13@ubuntu-24.04
- GitHub Check: 🧪 Tests / pytest@🐍3.9@windows-2025
- GitHub Check: 🧪 Tests / pytest@🐍3.9@ubuntu-24.04
- GitHub Check: 🧪 Tests / pytest@🐍3.10@windows-2025
- GitHub Check: 🧪 Tests / pytest@🐍3.13@windows-2025
- GitHub Check: Lint GitHub Actions / 🔐 Analyze repo with zizmor 🌈
There was a problem hiding this comment.
Pull Request Overview
This PR updates the GitHub Actions workflow for PR title validation to work with pull requests from forks by switching from pull_request to pull_request_target trigger and adding security configuration.
- Changed trigger from
pull_requesttopull_request_targetto enable fork compatibility - Added zizmor security linting ignore comment for the dangerous trigger
- Added
reopenedevent type to the workflow triggers
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
|
This PR is included in version 1.102.0 🎉 |
Suppress
zizmor's error: we do wantpull_request_target- https://github.com/amannn/action-semantic-pull-request#event-triggers