Skip to content

Add /ready slash command and author notification for "Author attention needed" label#4353

Merged
cheenamalhotra merged 7 commits into
mainfrom
dev/cheena/action-review-ready
Jun 11, 2026
Merged

Add /ready slash command and author notification for "Author attention needed" label#4353
cheenamalhotra merged 7 commits into
mainfrom
dev/cheena/action-review-ready

Conversation

@cheenamalhotra

@cheenamalhotra cheenamalhotra commented Jun 10, 2026

Copy link
Copy Markdown
Member

Summary

This PR adds two GitHub Actions workflows to streamline the review feedback loop:

1. Notify Author (notify-author-attention.yml)

When the "Author attention needed" label is added to a PR, a comment is automatically posted notifying the author to use /ready when they've addressed feedback.

2. Remove Label on /ready (remove-author-attention-label.yml)

When the PR author posts a /ready comment:

  • Removes the "Author attention needed" label
  • Re-requests reviews from all previous reviewers

Workflow

  1. Reviewer adds "Author attention needed" label → bot comments notifying the author
  2. Author addresses feedback and comments /ready → bot removes label and re-engages reviewers

Sample workflow: cheenamalhotra#9

Changes

  • Added .github/workflows/notify-author-attention.yml
  • Added .github/workflows/remove-author-attention-label.yml

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

Adds a GitHub Actions workflow to let PR authors comment /ready to remove the “Author Attention Needed” label and re-request reviews, helping move PRs back into the review queue without maintainer intervention.

Changes:

  • Introduces a new issue_comment-triggered workflow gated to PR-author comments containing /ready.
  • Removes the Author Attention Needed label when present.
  • Re-requests reviews from prior reviewers.

Comment thread .github/workflows/remove-author-attention-label.yml
Comment thread .github/workflows/remove-author-attention-label.yml Outdated
@cheenamalhotra cheenamalhotra mentioned this pull request Jun 10, 2026
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Copilot AI review requested due to automatic review settings June 10, 2026 18:09

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

Copilot reviewed 1 out of 1 changed files in this pull request and generated 2 comments.

Comment thread .github/workflows/remove-author-attention-label.yml Outdated
Comment thread .github/workflows/remove-author-attention-label.yml
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Copilot AI review requested due to automatic review settings June 10, 2026 18:31

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

Copilot reviewed 1 out of 1 changed files in this pull request and generated 1 comment.

Comment thread .github/workflows/remove-author-attention-label.yml
@cheenamalhotra cheenamalhotra moved this from To triage to In review in SqlClient Board Jun 10, 2026
@cheenamalhotra cheenamalhotra changed the title Add /ready slash command to remove "Author Attention Needed" label Add /ready slash command and author notification for "Author Attention Needed" label Jun 11, 2026
@cheenamalhotra cheenamalhotra added Author attention needed PRs that require author to respond or make updates to PR. and removed Author attention needed PRs that require author to respond or make updates to PR. labels Jun 11, 2026
Copilot AI review requested due to automatic review settings June 11, 2026 05:55
@cheenamalhotra cheenamalhotra added the Author attention needed PRs that require author to respond or make updates to PR. label Jun 11, 2026

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

Copilot reviewed 2 out of 2 changed files in this pull request and generated 4 comments.

Comment thread .github/workflows/notify-author-attention.yml
Comment thread .github/workflows/notify-author-attention.yml
Comment thread .github/workflows/remove-author-attention-label.yml Outdated
Comment thread .github/workflows/remove-author-attention-label.yml
@cheenamalhotra cheenamalhotra changed the title Add /ready slash command and author notification for "Author Attention Needed" label Add /ready slash command and author notification for "Author attention needed" label Jun 11, 2026
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Copilot AI review requested due to automatic review settings June 11, 2026 05:59

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

Copilot reviewed 2 out of 2 changed files in this pull request and generated 3 comments.

Comment thread .github/workflows/notify-author-attention.yml
Comment thread .github/workflows/remove-author-attention-label.yml Outdated
Comment thread .github/workflows/remove-author-attention-label.yml
@cheenamalhotra cheenamalhotra removed the Author attention needed PRs that require author to respond or make updates to PR. label Jun 11, 2026
@cheenamalhotra cheenamalhotra merged commit 1afe611 into main Jun 11, 2026
7 checks passed
@github-project-automation github-project-automation Bot moved this from In review to Done in SqlClient Board Jun 11, 2026
@cheenamalhotra cheenamalhotra deleted the dev/cheena/action-review-ready branch June 11, 2026 23:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

6 participants