chore: use shared semantic PR title workflow#630
Conversation
|
Warning Review the following alerts detected in dependencies. According to your organization's Security Policy, it is recommended to resolve "Warn" alerts. Learn more about Socket for GitHub.
|
Prompt To Fix All With AIFix the following 1 code review issue. Work through them one at a time, proposing concise fixes.
---
### Issue 1 of 1
.github/workflows/lint-pr.yml:21
The pinned SHA has no inline comment describing what commit or tag it resolves to. The removed action had `# pin v6.1.1` which made it easy to audit and bump the pin. Adding a similar comment here keeps the same traceability.
```suggestion
uses: PostHog/.github/.github/workflows/semantic-pr-title.yml@926dd076f0c796f7531177ae5cfcf1cf7cf0aeb3 # main as of 2026-05-27
```
Reviews (1): Last reviewed commit: "chore: use shared semantic PR title work..." | Re-trigger Greptile |
| - uses: amannn/action-semantic-pull-request@48f256284bd46cdaab1048c3721360e808335d50 # pin v6.1.1 | ||
| env: | ||
| GITHUB_TOKEN: ${{ github.token }} | ||
| uses: PostHog/.github/.github/workflows/semantic-pr-title.yml@926dd076f0c796f7531177ae5cfcf1cf7cf0aeb3 |
There was a problem hiding this comment.
The pinned SHA has no inline comment describing what commit or tag it resolves to. The removed action had
# pin v6.1.1 which made it easy to audit and bump the pin. Adding a similar comment here keeps the same traceability.
| uses: PostHog/.github/.github/workflows/semantic-pr-title.yml@926dd076f0c796f7531177ae5cfcf1cf7cf0aeb3 | |
| uses: PostHog/.github/.github/workflows/semantic-pr-title.yml@926dd076f0c796f7531177ae5cfcf1cf7cf0aeb3 # main as of 2026-05-27 |
Prompt To Fix With AI
This is a comment left during a code review.
Path: .github/workflows/lint-pr.yml
Line: 21
Comment:
The pinned SHA has no inline comment describing what commit or tag it resolves to. The removed action had `# pin v6.1.1` which made it easy to audit and bump the pin. Adding a similar comment here keeps the same traceability.
```suggestion
uses: PostHog/.github/.github/workflows/semantic-pr-title.yml@926dd076f0c796f7531177ae5cfcf1cf7cf0aeb3 # main as of 2026-05-27
```
How can I resolve this? If you propose a fix, please make it concise.Note: If this suggestion doesn't match your team's coding style, reply to this and let me know. I'll remember it for next time!
posthog-python Compliance ReportDate: 2026-05-27 18:50:35 UTC ✅ All Tests Passed!45/45 tests passed Capture Tests✅ 29/29 tests passed View Details
Feature_Flags Tests✅ 16/16 tests passed View Details
|
💡 Motivation and Context
PostHog/.githubnow provides a shared reusable workflow for validating PR titles against Conventional Commits. This switches the local SDK workflow to call that shared workflow, pinned to the merged.githubworkflow SHA, so the semantic PR title policy is centralized while each SDK repo still owns its PR trigger.This supports the changelog flow because merged PR titles are used to determine changelog entries.
💚 How did you test it?
📝 Checklist
If releasing new changes