Skip to content

Update auto-issue-comment workflow with author check#1300

Merged
rohan-pandeyy merged 1 commit into
AOSSIE-Org:mainfrom
Hemil36:refactor-auto-comment-workflow
Jun 1, 2026
Merged

Update auto-issue-comment workflow with author check#1300
rohan-pandeyy merged 1 commit into
AOSSIE-Org:mainfrom
Hemil36:refactor-auto-comment-workflow

Conversation

@Hemil36
Copy link
Copy Markdown
Contributor

@Hemil36 Hemil36 commented Jun 1, 2026

Add condition to skip comments for certain authors

Addressed Issues:

Fixes #1278

Screenshots/Recordings:

TODO: If applicable, add screenshots or recordings that demonstrate the interface before and after the changes.

Additional Notes:

AI Usage Disclosure:

We encourage contributors to use AI tools responsibly when creating Pull Requests. While AI can be a valuable aid, it is essential to ensure that your contributions meet the task requirements, build successfully, include relevant tests, and pass all linters. Submissions that do not meet these standards may be closed without warning to maintain the quality and integrity of the project. Please take the time to understand the changes you are proposing and their impact. AI slop is strongly discouraged and may lead to banning and blocking. Do not spam our repos with AI slop.

Check one of the checkboxes below:

  • This PR does not contain AI-generated code at all.
  • This PR contains AI-generated code. I have read the AI Usage Policy and this PR complies with this policy. I have tested the code locally and I am responsible for it.

I have used the following AI models and tools: TODO

Checklist

  • My PR addresses a single issue, fixes a single bug or makes a single improvement.
  • My code follows the project's code style and conventions
  • If applicable, I have made corresponding changes or additions to the documentation
  • If applicable, I have made corresponding changes or additions to tests
  • My changes generate no new warnings or errors
  • I have joined the Discord server and I will share a link to this PR with the project maintainers there
  • I have read the Contribution Guidelines
  • Once I submit my PR, CodeRabbit AI will automatically review it and I will address CodeRabbit's comments.
  • I have filled this PR template completely and carefully, and I understand that my PR may be closed without review otherwise.

Summary by CodeRabbit

  • Chores
    • Modified issue automation workflow to run automatic comments selectively—now restricted to issues from external contributors, excluding team members and project owners.

Add condition to skip comments for certain authors
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Jun 1, 2026

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: a9e44a27-2032-46d9-aadd-eacbb2da97a8

📥 Commits

Reviewing files that changed from the base of the PR and between 8f4b756 and 77268ca.

📒 Files selected for processing (1)
  • .github/workflows/auto-issue-comment.yml

Walkthrough

The auto-issue-comment GitHub Actions workflow now includes a conditional guard that prevents the automated welcome comment from posting when the issue author is an organization owner, member, or collaborator. The condition filters execution based on the issue author's association metadata before the job runs.

Changes

Auto-comment workflow maintainer gating

Layer / File(s) Summary
Conditional gate for maintainer issues
.github/workflows/auto-issue-comment.yml
The jobs.comment job gains an if: condition that gates execution based on github.event.issue.author_association, excluding OWNER, MEMBER, and COLLABORATOR associations, preventing redundant comments on internally created issues.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Suggested reviewers

  • rahulharpal1603
  • rohan-pandeyy

Poem

🐰 A simple gate keeps maintainers free,
No comment spam, just clarity.
When OWNER opens an issue bright,
The workflow skips—it knows what's right! 🌟

🚥 Pre-merge checks | ✅ 4
✅ Passed checks (4 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately summarizes the main change: adding an author check to the auto-issue-comment workflow to skip automated comments for certain authors.
Linked Issues check ✅ Passed The pull request successfully implements the requirement from issue #1278 by adding a conditional to skip posting automated comments when the issue author is an OWNER, MEMBER, or COLLABORATOR.
Out of Scope Changes check ✅ Passed All changes are directly related to the linked issue #1278 and implement only the specified requirement without introducing unrelated modifications.

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

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

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.

@rohan-pandeyy rohan-pandeyy merged commit ffd438a into AOSSIE-Org:main Jun 1, 2026
6 of 7 checks passed
@rohan-pandeyy
Copy link
Copy Markdown
Contributor

Thank you for your contribution @Hemil36!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Enhancement: Enhance auto issue comment workflow to skip maintainers

2 participants