Skip to content

Add needs-author-feedback GitHub Actions workflows#160

Merged
JacksonWeber merged 1 commit into
microsoft:mainfrom
JacksonWeber:feat/needs-author-feedback-workflows
May 29, 2026
Merged

Add needs-author-feedback GitHub Actions workflows#160
JacksonWeber merged 1 commit into
microsoft:mainfrom
JacksonWeber:feat/needs-author-feedback-workflows

Conversation

@JacksonWeber
Copy link
Copy Markdown
Contributor

Ports microsoft/opentelemetry-distro-python#178 to the JavaScript distro.

Adds two GitHub Actions workflows:

  • .github/workflows/needs-author-feedback.yml — When the needs-author-feedback label is applied to an issue, posts a notification comment to the author with an applied marker. When the issue author replies, the label is removed automatically.
  • .github/workflows/needs-author-feedback-followup.yml — Daily cron (03:00 UTC, also workflow_dispatch). Backfills missing markers, removes the label if the author has replied, posts a friendly reminder after 7 days, and closes the issue (removing the label first, so reopens get a fresh clock) after 14 days of no response.

Ports microsoft/opentelemetry-distro-python#178 to the JavaScript distro.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot AI review requested due to automatic review settings May 29, 2026 19:50
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR ports the “needs-author-feedback” automation from the Python distro to this JavaScript distro by introducing two GitHub Actions workflows that notify issue authors when more info is needed, then automatically follow up (remind / remove label / close) based on author responses and elapsed time.

Changes:

  • Add an issues/issue_comment-triggered workflow to comment with an applied marker when needs-author-feedback is labeled, and to remove the label when the issue author replies.
  • Add a scheduled/manual follow-up workflow that backfills missing applied markers, removes the label when the author replies, reminds after 7 days, and closes after 14 days.

Reviewed changes

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

File Description
.github/workflows/needs-author-feedback.yml Adds label-applied notification comment and auto-unlabel behavior on issue-author reply.
.github/workflows/needs-author-feedback-followup.yml Adds daily/manual follow-up automation to backfill markers, remind, remove label, and auto-close after timeouts.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment on lines +46 to +48
remove-label-on-author-reply:
if: github.event_name == 'issue_comment' && github.event.action == 'created'
runs-on: ubuntu-latest
Comment on lines +8 to +10
permissions:
issues: write

Comment on lines +100 to +116
try {
await github.rest.issues.removeLabel({
owner,
repo,
issue_number,
name: "needs-author-feedback"
});
} catch (error) {
if (error.status !== 404) throw error;
}

await github.rest.issues.createComment({
owner,
repo,
issue_number,
body: "Thanks for the response. Removing `needs-author-feedback`."
});
Comment on lines +65 to +81
try {
await github.rest.issues.removeLabel({
owner,
repo,
issue_number,
name: "needs-author-feedback"
});
} catch (error) {
if (error.status !== 404) throw error;
}

await github.rest.issues.createComment({
owner,
repo,
issue_number,
body: "Thanks for the response. Removing `needs-author-feedback`."
});
Comment on lines +142 to +146
try {
await github.rest.issues.removeLabel({
owner,
repo,
issue_number,
@JacksonWeber JacksonWeber merged commit 0457158 into microsoft:main May 29, 2026
5 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.

3 participants