Skip to content

Add GitHub Action to request RFC reviews from maintainers#15

Open
B-Step62 wants to merge 1 commit into
mainfrom
add-rfc-review-workflow
Open

Add GitHub Action to request RFC reviews from maintainers#15
B-Step62 wants to merge 1 commit into
mainfrom
add-rfc-review-workflow

Conversation

@B-Step62
Copy link
Copy Markdown
Contributor

@B-Step62 B-Step62 commented May 7, 2026

Summary

  • Adds .github/workflows/request-rfc-review.yml, a GitHub Action that fires on pull_request_target (opened / ready_for_review) when files under rfcs/** change.
  • Detects a new RFC by checking that the PR adds a markdown file matching rfcs/<slug>/<file>.md. Edits to existing RFCs are ignored.
  • Posts a single comment tagging the core MLflow maintainers (@dbczumar @daniellok-db @harupy @kriscon-db @mprahl @PattaraS @serena-ruan @TomeHirata @B-Step62) and asking for review. The PR author is excluded from the @mention list.
  • Idempotent: an HTML-comment marker prevents duplicate posts if the workflow re-fires (e.g. draft → ready_for_review).

Test plan

  • Open a test PR that adds a new rfcs/NNNN-test/NNNN-test.md and confirm the maintainer-tagging comment is posted exactly once.
  • Open a PR that only edits an existing RFC's markdown and confirm no comment is posted.
  • Open a draft PR adding a new RFC and confirm the comment is posted only after marking it ready for review.

This pull request was AI-assisted by Isaac.

Trigger on PRs that add a new RFC markdown file under rfcs/<slug>/
and post a comment tagging the core maintainers for review.

Co-authored-by: Isaac
marker,
`Thanks for opening a new RFC! ${mentionList} — please take a look when you have a chance and leave your feedback.`,
"",
"Reviewers: see the [RFC process](../blob/main/README.md) for guidance on what to look for.",
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

I don't think we can use relative links in the comments

Suggested change
"Reviewers: see the [RFC process](../blob/main/README.md) for guidance on what to look for.",
"Reviewers: see the [RFC process](https://github.com/mlflow/rfcs/blob/main/README.md) for guidance on what to look for.",

const prNumber = context.payload.pull_request.number;
const author = context.payload.pull_request.user.login.toLowerCase();

const maintainers = [
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Just a thought, have you considered creating a team in MLflow org that this workflow could tag or adding OWNERS file to the repo instead?

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