-
Notifications
You must be signed in to change notification settings - Fork 108
feat: Add automatic mentor assignment bot for new contributors #1067
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat: Add automatic mentor assignment bot for new contributors #1067
Conversation
- Added bot-mentor-assignment.yml workflow that triggers on issue assignment - Added mentor_assignment.sh script that: - Checks if issue has 'good first issue' label - Verifies assignee is a new contributor (no merged PRs) - Assigns random mentor from @hiero-ledger/hiero-sdk-python-triage team - Posts welcoming message with helpful resources Closes hiero-ledger#1063 Signed-off-by: Aditya Shirsatrao <adityashirsatrao007@gmail.com> Signed-off-by: adityashirsatrao007 <adityashirsatrao007@gmail.com>
There was a problem hiding this 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 implements an automatic mentor assignment bot that welcomes new contributors working on "good first issue" labeled issues and assigns them a mentor from the triage team.
Key Changes:
- Added GitHub Actions workflow that triggers on issue assignment events
- Implemented bash script to verify new contributors, fetch mentors, and post welcome messages
- Updated CHANGELOG to document the new feature
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 8 comments.
| File | Description |
|---|---|
| CHANGELOG.md | Added entry documenting the new mentor assignment bot feature |
| .github/workflows/bot-mentor-assignment.yml | New workflow that triggers on issue assignments and executes the mentor assignment script |
| .github/scripts/mentor_assignment.sh | New script that checks for "good first issue" label, verifies new contributor status, assigns a mentor, and posts a welcome message |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Signed-off-by: Aditya Shirsatrao <adityashirsatrao007@gmail.com> Signed-off-by: adityashirsatrao007 <adityashirsatrao007@gmail.com>
exploreriii
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi @adityashirsatrao007
I will not accept this PR until its tested on your fork and assignments are done on your fork please (not posted here)
- Added workflow_dispatch trigger with dry_run input for testing - Added DRY_RUN mode that logs without posting comments - Added concurrency group to prevent duplicate messages - Added members:read permission for team API access - Added set -euo pipefail for safer script execution - Fixed jq command (removed @JSON to avoid double-encoding) - Consolidated TRIAGE_TEAM variables for maintainability - Improved duplicate detection using regex pattern - Removed threads.json file Signed-off-by: Aditya Shirsatrao <adityashirsatrao007@gmail.com> Signed-off-by: adityashirsatrao007 <adityashirsatrao007@gmail.com>
|
Hi, I've addressed all the review feedback: Changes made:
Testing:
The dry_run mode will show what would be posted without actually commenting. |
|
This is not correct enable them in your repository settings |
|
This is untested thus not ready for review |
|
Hi @adityashirsatrao007 Complete your two other PRs, please,or, request to be removed from them |

Description
This PR implements the automatic mentor assignment feature for new contributors, as proposed in #1063.
What's Added
New Workflow: .github/workflows/bot-mentor-assignment.yml\
New Script: .github/scripts/mentor_assignment.sh\
Welcome Message Features
The bot posts a friendly welcome message that includes:
Testing
This workflow follows the same patterns as existing bot workflows (\�ot-assignment-check.yml, \�ot-workflows.yml) and can be tested using the fork testing workflow as documented in \docs/sdk_developers/training/testing_forks.md.
Closes
Closes #1063
Checklist