chore: cleanup unused common workflows and scripts#710
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (6)
💤 Files with no reviewable changes (6)
WalkthroughSix files are deleted: the Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Possibly related PRs
Suggested reviewers
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
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 |
There was a problem hiding this comment.
Pull request overview
Removes reusable GitHub Actions workflows and helper scripts from this repository now that they’ve been centralized in openfga/.github, keeping the SDK generator repo focused on generation/testing concerns.
Changes:
- Deleted reusable release-related workflows (
release-please,undraft-release) and their parsing/test scripts. - Removed the PR title conventional-commit reusable workflow.
- Removed the workflow that unit-tested the release parsing scripts (since the scripts are no longer in this repo).
Reviewed changes
Copilot reviewed 6 out of 6 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
.github/workflows/undraft-release.yml |
Removed reusable undraft-release workflow (centralized elsewhere). |
.github/workflows/test-release-scripts.yml |
Removed CI workflow for release script tests (scripts removed). |
.github/workflows/scripts/parse-release.test.sh |
Removed release parsing unit test script. |
.github/workflows/scripts/parse-release.sh |
Removed release parsing helper script. |
.github/workflows/release-please.yml |
Removed reusable release-please workflow previously hosted here. |
.github/workflows/pr-title-check.yml |
Removed reusable PR title conventional-commit checker workflow. |
Description
The common workflows and scripts are now removed from the SDK generator and moved to https://github.com/openfga/.github so these are now all safe to delete.
What problem is being solved?
How is it being solved?
What changes are made to solve it?
References
Review Checklist
mainSummary by CodeRabbit