Skip to content

chore: cleanup unused common workflows and scripts#710

Merged
SoulPancake merged 5 commits into
mainfrom
chore/cleanup-unused-workflows
Jun 15, 2026
Merged

chore: cleanup unused common workflows and scripts#710
SoulPancake merged 5 commits into
mainfrom
chore/cleanup-unused-workflows

Conversation

@SoulPancake

@SoulPancake SoulPancake commented Jun 15, 2026

Copy link
Copy Markdown
Member

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

  • I have clicked on "allow edits by maintainers".
  • I have added documentation for new/changed functionality in this PR or in a PR to openfga.dev [Provide a link to any relevant PRs in the references section above]
  • The correct base branch is being used, if not main
  • I have added tests to validate that the change in functionality is working as expected

Summary by CodeRabbit

  • Chores
    • Removed PR title validation workflow.
    • Removed release automation workflows and associated helper scripts.
    • Removed release script testing infrastructure.

@SoulPancake SoulPancake requested a review from a team as a code owner June 15, 2026 14:27
Copilot AI review requested due to automatic review settings June 15, 2026 14:27
@coderabbitai

coderabbitai Bot commented Jun 15, 2026

Copy link
Copy Markdown
Contributor

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: fe337e43-a6aa-4b59-9a59-4e300f35f7e9

📥 Commits

Reviewing files that changed from the base of the PR and between e2197d0 and 520e761.

📒 Files selected for processing (6)
  • .github/workflows/pr-title-check.yml
  • .github/workflows/release-please.yml
  • .github/workflows/scripts/parse-release.sh
  • .github/workflows/scripts/parse-release.test.sh
  • .github/workflows/test-release-scripts.yml
  • .github/workflows/undraft-release.yml
💤 Files with no reviewable changes (6)
  • .github/workflows/pr-title-check.yml
  • .github/workflows/undraft-release.yml
  • .github/workflows/test-release-scripts.yml
  • .github/workflows/scripts/parse-release.test.sh
  • .github/workflows/scripts/parse-release.sh
  • .github/workflows/release-please.yml

Walkthrough

Six files are deleted: the pr-title-check.yml, release-please.yml, undraft-release.yml, and test-release-scripts.yml GitHub Actions workflows, along with the parse-release.sh helper script and its parse-release.test.sh test harness. No replacement files are introduced.

Changes

Cohort / File(s) Summary
Release Automation Removal
.github/workflows/pr-title-check.yml, .github/workflows/release-please.yml, .github/workflows/undraft-release.yml, .github/workflows/test-release-scripts.yml, .github/workflows/scripts/parse-release.sh, .github/workflows/scripts/parse-release.test.sh
Removes all custom release automation: the reusable workflow_call workflows for PR title validation, release-please orchestration (with GPG-signed tags, release drafting, and branch sync), undrafting releases, and the test workflow; also removes the parse-release.sh script (manifest-diff and changelog-notes subcommands) and its full Bash test harness.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

Suggested reviewers

  • rhamzeh
🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly and concisely summarizes the main change: removal of unused workflows and scripts that were migrated to a centralized repository.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch chore/cleanup-unused-workflows

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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.

@SoulPancake SoulPancake added this pull request to the merge queue Jun 15, 2026
Merged via the queue into main with commit bff6790 Jun 15, 2026
13 of 16 checks passed
@SoulPancake SoulPancake deleted the chore/cleanup-unused-workflows branch June 15, 2026 14:31
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