Skip to content

Remove unused workflows#16

Merged
mprahl merged 1 commit into
opendatahub-io:masterfrom
mprahl:remove-workflows
Nov 22, 2025
Merged

Remove unused workflows#16
mprahl merged 1 commit into
opendatahub-io:masterfrom
mprahl:remove-workflows

Conversation

@mprahl

@mprahl mprahl commented Nov 22, 2025

Copy link
Copy Markdown
🛠 DevTools 🛠

Open in GitHub Codespaces

Install mlflow from this PR

# mlflow
pip install git+https://github.com/mlflow/mlflow.git@refs/pull/16/merge
# mlflow-skinny
pip install git+https://github.com/mlflow/mlflow.git@refs/pull/16/merge#subdirectory=libs/skinny

For Databricks, use the following command:

%sh curl -LsSf https://raw.githubusercontent.com/mlflow/mlflow/HEAD/dev/install-skinny.sh | sh -s pull/16/merge

These workflows are not relevant on forks.

Summary by CodeRabbit

Chores

  • Removed multiple GitHub Actions automation workflows for pull request management, auto-formatting, release automation, and code review processes.
  • Simplified repository configuration and documentation to reflect removal of workflow automation.

✏️ Tip: You can customize this high-level summary in your review settings.

@github-actions

Copy link
Copy Markdown

@mprahl Thank you for the contribution! Could you fix the following issue(s)?

⚠ Invalid PR template

This PR does not appear to have been filed using the MLflow PR template. Please copy the PR template from here and fill it out.

@coderabbitai

coderabbitai Bot commented Nov 22, 2025

Copy link
Copy Markdown

Caution

Review failed

The pull request is closed.

Walkthrough

This pull request removes the PR template documentation and discontinues multiple GitHub Actions workflows, including automation for code formatting, labeling, release notes, approval checks, issue closing, cherry-pick warnings, image publishing, code review integration, and resolution assistance. All related JavaScript helper modules are also deleted.

Changes

Cohort / File(s) Summary
PR Template & Workflow Documentation
pull_request_template.md, workflows/README.md
Simplifies PR template by removing documentation, release notes, and classification sections; removes automation table from README with note on omitted governance workflows
Advice Workflow
workflows/advice.js, workflows/advice.yml
Deletes workflow automation that validated PR templates, checked DCO compliance, provided Codespaces instructions, and notified authors of issues
Autoformat Workflow
workflows/autoformat.js, workflows/autoformat.yml
Removes workflow orchestrating PR autoformat checks including status updates, workflow approvals, and maintainer permission validation
Cancel Workflow
workflows/cancel.js, workflows/cancel.yml
Deletes automation that cancelled unfinished workflow runs associated with closed/merged PRs
Cherry-picks Warning Workflow
workflows/cherry-picks-warn.yml
Removes workflow notifying authors on release branches about cherry-pick merge practices
Closing PR Workflow
workflows/closing-pr.js, workflows/closing-pr.yml
Deletes automation that parsed closing syntax in PR bodies and labeled related issues
Labeling Workflow
workflows/labeling.js, workflows/labeling.yml
Removes workflow that extracted labels from PR/issue bodies and synchronized them with GitHub issue labels
Maintainer Approval Workflows
workflows/maintainer-approval.yml, workflows/require-core-maintainer-approval.js
Deletes automation enforcing core maintainer approval on PRs
Patch Labeling Workflow
workflows/patch.js, workflows/patch.yml
Removes semantic versioning and patch release labeling automation
Release & Image Workflows
workflows/push-images.yml, workflows/release-note.js, workflows/release-note.yml, workflows/update-release-labels.yml
Deletes release note validation, post-merge remediation, Docker image building/publishing, and release label update workflows
Code Review & Resolution Workflows
workflows/resolve.yml, workflows/review.yml, workflows/team-review.yml
Removes Claude AI code review integration, PR resolution assistance, and automated team reviewer assignment workflows

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~25 minutes

  • Multiple complex workflows: Several removed workflows contain intricate control flow (resolve.yml, review.yml, patch.js) requiring verification of intentional removal
  • High file count: 17+ files across related automation systems necessitate systematic verification
  • Public API removal: Numerous exported functions and module exports across JavaScript helpers require confirmation no other code depends on them
  • Workflow interconnections: Some workflows potentially reference or depend on each other (e.g., release-note, patch labeling, approval checks)

Poem

🐰 A chorus of workflows takes its final bow,
No more formatting nudges or approval vows,
The rabbit bids them farewell with a knowing hop,
Cleaner workflows emerge where the chaos did stop! 🎭✨

✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 2b8d8ff and b05a891.

📒 Files selected for processing (24)
  • .github/pull_request_template.md (0 hunks)
  • .github/workflows/README.md (1 hunks)
  • .github/workflows/advice.js (0 hunks)
  • .github/workflows/advice.yml (0 hunks)
  • .github/workflows/autoformat.js (0 hunks)
  • .github/workflows/autoformat.yml (0 hunks)
  • .github/workflows/cancel.js (0 hunks)
  • .github/workflows/cancel.yml (0 hunks)
  • .github/workflows/cherry-picks-warn.yml (0 hunks)
  • .github/workflows/closing-pr.js (0 hunks)
  • .github/workflows/closing-pr.yml (0 hunks)
  • .github/workflows/labeling.js (0 hunks)
  • .github/workflows/labeling.yml (0 hunks)
  • .github/workflows/maintainer-approval.yml (0 hunks)
  • .github/workflows/patch.js (0 hunks)
  • .github/workflows/patch.yml (0 hunks)
  • .github/workflows/push-images.yml (0 hunks)
  • .github/workflows/release-note.js (0 hunks)
  • .github/workflows/release-note.yml (0 hunks)
  • .github/workflows/require-core-maintainer-approval.js (0 hunks)
  • .github/workflows/resolve.yml (0 hunks)
  • .github/workflows/review.yml (0 hunks)
  • .github/workflows/team-review.yml (0 hunks)
  • .github/workflows/update-release-labels.yml (0 hunks)

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.

These workflows are not relevant on forks.

Signed-off-by: mprahl <mprahl@users.noreply.github.com>
@mprahl mprahl merged commit 3f84104 into opendatahub-io:master Nov 22, 2025
27 of 34 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.

2 participants